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

[NSE-207] Fix aggregate and refresh UT test script #426

Merged
merged 5 commits into from
Jul 30, 2021

Conversation

rui-mo
Copy link
Collaborator

@rui-mo rui-mo commented Jul 23, 2021

What changes were proposed in this pull request?

This pr:

  • fixed aggregate and added fallback logic to window
  • refreshed UT test script

#207

How was this patch tested?

verified on TPC-DS v1 and v2

@github-actions
Copy link

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?

[NSE-${ISSUES_ID}] ${detailed message}

See also:

@rui-mo rui-mo force-pushed the wip_enable_ut branch 10 times, most recently from 75b835b to 2d45f67 Compare July 28, 2021 15:24
@rui-mo rui-mo changed the title [DNM][NSE-207] For enabling UT test [NSE-207] Fix aggregate and refresh UT test script Jul 29, 2021
@github-actions
Copy link

#207

@zhouyuan zhouyuan merged commit 06bd109 into oap-project:master Jul 30, 2021
zhouyuan added a commit that referenced this pull request Aug 5, 2021
* Minor: Remove debug code for PR#387 (#431)

* [NSE-436] Fix for Arrow Data Source test suite (#437)

Closes #436

* [NSE-254]Solve the redundant arrow library issue (#440)

* [NSE-254]Issue0410 jar size (#441)

* [NSE-254]Solve the redundant arrow library issue

* Remove mvn jar plugin 3.2.0

* fix packaging (#442)

Signed-off-by: Yuan Zhou <[email protected]>

Co-authored-by: Yuan Zhou <[email protected]>

* [NSE-207] Fix aggregate and refresh UT test script (#426)

* fix expr id difference in Partial Aggregate

* add fallback to window

* refresh ut

* use stol instead of stoi

* enable ut full test

* [NSE-429] TPC-DS Q14a/b get slowed down within setting spark.oap.sql.columnar.sortmergejoin.lazyread=true (#432)

* wip

* debug commit

* wip

* discard proxy pattern

* fix

* fix

* fix

* fix unexpected O(n^2) time

* fix1

* fix2

* Add a bunch of caches; Adjust prefetch batch length to 16

* format

* fix

* disable spark.oap.sql.columnar.sortmergejoin.lazyread by default

* fix

Co-authored-by: Hongze Zhang <[email protected]>
Co-authored-by: Wei-Ting Chen <[email protected]>
Co-authored-by: Rui Mo <[email protected]>
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.

2 participants