-
Notifications
You must be signed in to change notification settings - Fork 447
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] Add config for show velox task metrics when finished #6573
Conversation
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?
See also: |
Run Gluten Clickhouse CI |
Can we use kDebugModeEnabled? |
kDebugModeEnabled still works but it's a coarse-grained config, new config aims to show more details for production job. |
Run Gluten Clickhouse CI |
cc @ulysses-you @liujiayi771 thank you! |
* [GLUTEN-7024][VL] Skip call collectMetrics when the task does not call next() (#7025) * [GLUTEN-7130][CORE] Skip command execution when collect qe fallback summary (#7132) * [VL] Add config for show velox task metrics when finished (#6573) --------- Co-authored-by: zhaokuo <[email protected]> Co-authored-by: Zhen Wang <[email protected]> Co-authored-by: Yang Zhang <[email protected]>
What changes were proposed in this pull request?
Add config
spark.gluten.sql.columnar.backend.velox.showTaskMetricsWhenFinished
for show velox task metrics when finished, it helps debug task level execution.