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.

Sign up for GitHub

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

Jump to bottom

[BUG] Remove duplicated executor CPU time metric from SQLTaskAggMetricsProfileResult #1504

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

[BUG] Remove duplicated executor CPU time metric from SQLTaskAggMetricsProfileResult #1504

wants to merge 2 commits into from
+54 −31

Conversation

cindyyuanjiang
Copy link
Collaborator

Fixes #1467

Changes

  • Removed duplicated executor CPU time metric from SQLTaskAggMetricsProfileResult
  • Updated existing unit tests

@cindyyuanjiang cindyyuanjiang self-assigned this Jan 19, 2025
@cindyyuanjiang cindyyuanjiang added bug Something isn't working core_tools Scope the core module (scala) affect-output A change that modifies the output (add/remove/rename files, add/remove/rename columns) labels Jan 19, 2025
@cindyyuanjiang
updated header
0b5dca9 
Signed-off-by: cindyyuanjiang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amahussein amahussein Awaiting requested review from amahussein

@parthosa parthosa Awaiting requested review from parthosa

@nartal1 nartal1 Awaiting requested review from nartal1

At least 1 approving review is required to merge this pull request.

Assignees

@cindyyuanjiang cindyyuanjiang

Labels
affect-output A change that modifies the output (add/remove/rename files, add/remove/rename columns) bug Something isn't working core_tools Scope the core module (scala)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] SQL/Task Aggregated Metrics contain duplicated executor CPU time results
1 participant
@cindyyuanjiang