-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Add more information in our select docs #7177
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7177 +/- ##
==========================================
+ Coverage 49.27% 49.29% +0.01%
==========================================
Files 288 288
Lines 44971 44971
==========================================
+ Hits 22161 22168 +7
+ Misses 20782 20778 -4
+ Partials 2028 2025 -3
Continue to review full report at Codecov.
|
docs/select/README.md
Outdated
- Large numbers (more than 64-bit) are not yet supported | ||
- Date related functions are not yet supported (EXTRACT, DATE_DIFF, etc) | ||
- S3's reserved keywords list is not yet respected | ||
Type inference and automatic conversion of values is performed based on the context when the value is un-typed (such as when reading CSV data). CAST function if present overrides automatic conversion. |
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.
For any reference to any API, or function of an API, should we be using a format like CAST
or Select
, or will regular formatting will suffice?
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.
This is a select doc so it refers to only select, I don't think we need to be explicit about. Since the context is implicit.
@donatello @eco-minio PTAL |
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.
LGTM
Changed strings to string PTAL @eco-minio @donatello |
Mint Automation
|
Description
Add more information in our select docs
Motivation and Context
Adding more details about our implementation
Regression
No
How Has This Been Tested?
Just documentation changes, need to be proof read.
Types of changes
Checklist:
mint
PR # here: )