Skip to content

Conversation

acvictor
Copy link
Contributor

@acvictor acvictor commented Aug 1, 2025

What changes were proposed in this pull request?

Rewrite disabled test in "Fallback Parquet V2 to V1" GlutenFileDataSourceV2FallBackSuite to check for Gluten operator.

How was this patch tested?

Added UT.

@github-actions github-actions bot added the CORE works for Gluten Core label Aug 1, 2025
Copy link

github-actions bot commented Aug 1, 2025

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

github-actions bot commented Aug 1, 2025

Run Gluten Clickhouse CI on x86

Copy link

github-actions bot commented Aug 1, 2025

Run Gluten Clickhouse CI on x86

Copy link

github-actions bot commented Aug 1, 2025

Run Gluten Clickhouse CI on x86

Copy link

github-actions bot commented Aug 2, 2025

Run Gluten Clickhouse CI on x86

@acvictor acvictor changed the title [WIP] [VL] Rewrite disabled test in GlutenFileDataSourceV2FallBackSuite Aug 2, 2025
@acvictor acvictor marked this pull request as ready for review August 2, 2025 16:16
@acvictor
Copy link
Contributor Author

acvictor commented Aug 2, 2025

Hi @rui-mo, can you please review this PR? The Velox test failure looks unrelated. Thanks!

Copy link
Contributor

@rui-mo rui-mo left a comment

Choose a reason for hiding this comment

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

Could you please take a look if the Clickhouse issue is relevant? Thanks.

@acvictor
Copy link
Contributor Author

acvictor commented Aug 6, 2025

Could you please take a look if the Clickhouse issue is relevant? Thanks.

@rui-mo it looks unrelated
image
image

@rui-mo
Copy link
Contributor

rui-mo commented Aug 6, 2025

Would you like to try if rebase could help clear the failure?

Copy link

github-actions bot commented Aug 6, 2025

Run Gluten Clickhouse CI on x86

Copy link

github-actions bot commented Aug 7, 2025

Run Gluten Clickhouse CI on x86

@acvictor
Copy link
Contributor Author

acvictor commented Aug 8, 2025

Would you like to try if rebase could help clear the failure?

@rui-mo there still seems to be some flakiness. Any suggestions?

@rui-mo
Copy link
Contributor

rui-mo commented Aug 8, 2025

Hi @zzcclp @baibaichen, do you have any clue for the CH failure? Thanks.

@rui-mo
Copy link
Contributor

rui-mo commented Aug 11, 2025

Hi @zhouyuan, do you have an impression about the failure of CH workflow? Thanks.

Copy link

Run Gluten Clickhouse CI on x86

@zhouyuan
Copy link
Contributor

As far as i know the jenkins of CK backend is not very stable these days. re-try to check.

@rui-mo rui-mo merged commit 961e17a into apache:main Aug 12, 2025
93 of 94 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CORE works for Gluten Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants