-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
finish adding timestamp_ns/ms/sec/tz datatypes #2506
Conversation
8577746
to
79179c5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2506 +/- ##
==========================================
+ Coverage 91.56% 92.87% +1.30%
==========================================
Files 1029 1025 -4
Lines 38059 38342 +283
==========================================
+ Hits 34849 35609 +760
+ Misses 3210 2733 -477 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may also have to add the timestamp support to scanPandas
46a5943
to
91ce18d
Compare
bdd48f9
to
cec0ed3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed changes on Node.js, Python, C, and Java APIs. LGTM
Added support for timestamp_ns/_ms/_sec/_tz datatypes