This repository has been archived by the owner on Sep 18, 2023. It is now read-only.
[NSE-239] Adopt ARROW-7011 #240
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for opening a pull request! Could you open an issue for this pull request on Github Issues? https://github.com/oap-project/native-sql-engine/issues Then could you also rename commit message and pull request title in the following format? See also: |
|
|
|
|
|
looks like the new cast is more efficient, local tests showed ~2% improvement on TPC-DS power |
zhouyuan
reviewed
Apr 9, 2021
| @@ -40,13 +40,7 @@ T round2(T val, int precision = 2) { | |||
| } | |||
There was a problem hiding this comment.
looks like this round2 func can also utilize below new cast method?
There was a problem hiding this comment.
Is arrow::Decimal128 stored internally using some char-like stuffs? I didn't see a "Reprecision" method or something is provided so I think it might be OK for just now to use string formatters. By the way I see there is another function std::setprecision might be easier to use than snprintf.
zhouyuan
approved these changes
Apr 12, 2021
Closed
zhouyuan
pushed a commit
to zhouyuan/native-sql-engine
that referenced
this pull request
Apr 15, 2021
zhouyuan
added a commit
that referenced
this pull request
Apr 19, 2021
* [NSE-229] Fix the deprecated code warning in shuffle_split_test (#230) * fix the deprecated code warning in shuffle_split_test * fix the code style * format update * [NSE-239] Adopt ARROW-7011 (#240) * [NSE-224] update third party code (#242) * update third party code Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-176]Arrow install order issue (#231) * Add Arrow install script * [NSE-176]Add Arrow install Script in Arrow Data Source * remove popd parameter for ubuntu * Change variables to BFS * Update Arrow Header and Find Arrow function * Fix one issue with reading wrong Arrow Path * Update ARROW_CSV=ON * Update build_arrow=OFF in Scala UT * Install parent pom file in Scala UT * [NSE-196] clean up native sql options (#215) * clean up native sql options Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding more options Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding more options Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding warning log for running on non-intel cpu Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-206]doc update on feature support status (#253) * update operators support status Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * update docs on operators supporting status Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-241] fix hashagg result length (#249) * fix hashagg result length Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * optimize on getting batch size Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * using fixed sized output len for hashagg Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-248] fix arrow dependency order (#259) * Only read .so.300.0.0 * Fix arroow dataset dependency issue * Add ARROW_S3=ON, Add symlink copy in CMakeList. Co-authored-by: JiaKe <ke.a.jia@intel.com> Co-authored-by: Hongze Zhang <hongze.zhang@intel.com> Co-authored-by: Wei-Ting Chen <weiting.chen@intel.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.