Skip to content
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

[VL] Enable Window oom in ci job #4929

Merged
merged 1 commit into from
Apr 10, 2024
Merged

[VL] Enable Window oom in ci job #4929

merged 1 commit into from
Apr 10, 2024

Conversation

JkSelf
Copy link
Contributor

@JkSelf JkSelf commented Mar 12, 2024

What changes were proposed in this pull request?

facebookincubator/velox#9025

How was this patch tested?

Existing tests

Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

1 similar comment
Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

@JkSelf JkSelf force-pushed the rank-test branch 5 times, most recently from f28bc88 to 3fb03f0 Compare April 1, 2024 11:50
@JkSelf JkSelf changed the title [DNM] Velox test [DNM] Test window optimization in Velox Apr 2, 2024
@JkSelf JkSelf force-pushed the rank-test branch 2 times, most recently from 8d1d065 to 12db9a3 Compare April 8, 2024 03:53
@JkSelf
Copy link
Contributor Author

JkSelf commented Apr 8, 2024

@zhztheplayer The CI is passed. Please help to pick this commit into oap velox in next round rebase. Thanks.

@zhztheplayer
Copy link
Member

Committed Velox change in https://github.com/oap-project/velox/tree/2024_04_09

Copy link
Member

@zhztheplayer zhztheplayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@JkSelf JkSelf changed the title [DNM] Test window optimization in Velox [VL] Enable Window oom in ci job Apr 10, 2024
@JkSelf JkSelf merged commit 8ab9b01 into apache:main Apr 10, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants