Skip to content

[CORE] Post events until both spark ui and gluten ui are enable#9272

Merged
jackylee-ch merged 1 commit intoapache:mainfrom
yikf:post-event
Apr 14, 2025
Merged

[CORE] Post events until both spark ui and gluten ui are enable#9272
jackylee-ch merged 1 commit intoapache:mainfrom
yikf:post-event

Conversation

@yikf
Copy link
Copy Markdown
Contributor

@yikf yikf commented Apr 9, 2025

What changes were proposed in this pull request?

Post events until both spark ui and gluten ui are enable. Prevents invalid gluten event post when the spark ui is not turned on

How was this patch tested?

GA

@github-actions github-actions bot added the CORE works for Gluten Core label Apr 9, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 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:

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2025

Run Gluten Clickhouse CI on x86

@yikf
Copy link
Copy Markdown
Contributor Author

yikf commented Apr 11, 2025

@jackylee-ch Could you please take a look.

Comment thread gluten-ui/src/main/scala/org/apache/spark/sql/execution/ui/GlutenUIUtils.scala Outdated
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@yikf yikf requested a review from jackylee-ch April 14, 2025 02:26
@jackylee-ch jackylee-ch merged commit 4dcd280 into apache:main Apr 14, 2025
47 checks passed
@jackylee-ch
Copy link
Copy Markdown
Contributor

Thanks for your work! @yikf

@yikf yikf deleted the post-event branch April 14, 2025 07:29
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.

2 participants