Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-119]allow to config batchsize in hashagg and wscg #222

Merged
merged 1 commit into from
Apr 2, 2021

Conversation

zhouyuan
Copy link
Collaborator

@zhouyuan zhouyuan commented Apr 1, 2021

What changes were proposed in this pull request?

This patch fixes hashagg and wscg to use correct batch size
fixes: #119
Signed-off-by: Yuan Zhou yuan.zhou@intel.com

How was this patch tested?

locally verified

@github-actions
Copy link

github-actions bot commented Apr 1, 2021

#119

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
@github-actions
Copy link

github-actions bot commented Apr 1, 2021


@zhouyuan zhouyuan merged commit c52cc1b into oap-project:master Apr 2, 2021
zhouyuan added a commit to zhouyuan/native-sql-engine that referenced this pull request Apr 2, 2021
Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
zhouyuan added a commit that referenced this pull request Apr 2, 2021
* [NSE-185] Avoid unnecessary copying when simply projecting on fields (#187)

* [NSE-185] Avoid unnecessary copying when simply projecting on fields

* Avoid sharing buffers in output

* [NSE-211] IndexOutOfBoundsException during running TPC-DS Q2 (#212)

Closes #211

* fix missing decimal check (#219)

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* [NSE-227]fix issues from codescan (#225)

* fix code scan

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* fix code scan

* fix format

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* fix wrong patch

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* Klockwork issues fix

* fix format

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* more fixes

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

Co-authored-by: Rui Mo <rui.mo@intel.com>
Co-authored-by: Hongze Zhang <hongze.zhang@intel.com>

* allow to config batchsize in hashagg and wscg (#222)

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

Co-authored-by: Hongze Zhang <hongze.zhang@intel.com>
Co-authored-by: Rui Mo <rui.mo@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

consolidate batch size
1 participant