This repository has been archived by the owner on Sep 18, 2023. It is now read-only.
[NSE-170] using unsafe appender #203
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
Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
|
|
Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
zhouyuan
added a commit
to zhouyuan/native-sql-engine
that referenced
this pull request
Mar 29, 2021
This patch adds an unsafe appender which will reserve space before builder array. The boolean builder is not touched as only malloc small sized memory The string builder are not touched as it's diffcult to pre-allocate the space * using unsafe appender Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
|
|
zhouyuan
added a commit
that referenced
this pull request
Mar 29, 2021
* [NSE-130] support decimal round and abs (#166) * support decimal round and abs * remove duplicate cast in multiply/divide * [NSE-161] adding format check (#165) * adding format check Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * formating code Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding google format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * reformat with new style Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * lower clang version to 10 Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding script to format code Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-170]improve sort shuffle code (#171) * improve sort shuffle code Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * pass by ref in builder Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix string/decimal builder Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-62]Fixing issue0062 for package arrow dependencies in jar with refresh2 (#172) * Add arrow build and dependency support * Add compress.sh default value * Fix bug for parameter's default value * Add CACHE PATH * fix copy bitmap in InplaceSort (#174) * [NSE-153] fix window results (#175) * fix window sort memory Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * remove unused code Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix windown w/o avg Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix decimal sort Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * Fix issue 179 for arrow include directory (#181) * Fix issue0191 for .so file copy to tmp. (#192) * Following NSE-153, optimize fallback conditions for columnar window (#189) * Fix q14a/b segfault (#193) Signed-off-by: Chendi Xue <chendi.xue@intel.com> * [NSE-194]Turn on several Arrow parameters (#195) * Turn on several Arrow parameters * Change SIMD Level Setting * Hashmap build opt for semi/anti/exists join (#197) Signed-off-by: Chendi Xue <chendi.xue@intel.com> * [NSE-198] support the month() and dayofmonth() functions with DateType (#199) * [NSE-206] fix doc link, update limitations (#205) * fix doc link Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * update arrow data source Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * adding limitations Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * mention limits Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * [NSE-170] using unsafe appender (#203) This patch adds an unsafe appender which will reserve space before builder array. The boolean builder is not touched as only malloc small sized memory The string builder are not touched as it's diffcult to pre-allocate the space * using unsafe appender Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * fix format Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> * update arrow branch Signed-off-by: Yuan Zhou <yuan.zhou@intel.com> Co-authored-by: Rui Mo <rui.mo@intel.com> Co-authored-by: Wei-Ting Chen <weiting.chen@intel.com> Co-authored-by: Hongze Zhang <hongze.zhang@intel.com> Co-authored-by: Chendi.Xue <chendi.xue@intel.com> Co-authored-by: JiaKe <ke.a.jia@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.
What changes were proposed in this pull request?
This patch adds an unsafe appender which will reserve space before builder array.
The boolean builder is not touched as only malloc small sized memory
The string builder are not touched as it's diffcult to pre-allocate the space
On my 2node env this can improve TPC-DS performance by ~5%
This approach is applied to Sort operator only.
related: #170
Signed-off-by: Yuan Zhou yuan.zhou@intel.com
How was this patch tested?
locally verified