-
Couldn't load subscription status.
- Fork 28.9k
[SPARK-26327][SQL][BACKPORT-2.4] Bug fix for FileSourceScanExec metrics update
#23287
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
Conversation
|
Test build #99976 has finished for PR 23287 at commit
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM. Merged to branch-2.4
…rics update ## What changes were proposed in this pull request? Backport #23277 to branch 2.4 without the metrics renaming. ## How was this patch tested? New test case in `SQLMetricsSuite`. Closes #23287 from xuanyuanking/SPARK-26327-2.4. Authored-by: Yuanjian Li <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
|
Hmm. It seems out of sync between GitBox and GitHub again. |
|
I triggered re-sync at https://selfserve.apache.org/ and am waiting. |
|
Hi, @srowen . If Apache SelfServe doesn't work, should we file an INFRA JIRA issue or do we need to wait for the next commit on |
|
I'd say either just wait today, or push a dummy commit to try again, and if those don't work, yes file an INFRA ticket. Maybe somehow the other branches aren't synced but I imagine they are supposed to. |
|
Thank you. Yep. I'll wait and monitor today and proceed like that. |
|
Thanks Sean and Dongjoon, just ping me if a dummy commit is needed. |
|
Thanks. @xuanyuanking . This landed on |
|
Thanks Dongjoon. |
…rics update ## What changes were proposed in this pull request? Backport apache#23277 to branch 2.4 without the metrics renaming. ## How was this patch tested? New test case in `SQLMetricsSuite`. Closes apache#23287 from xuanyuanking/SPARK-26327-2.4. Authored-by: Yuanjian Li <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
…rics update ## What changes were proposed in this pull request? Backport apache#23277 to branch 2.4 without the metrics renaming. ## How was this patch tested? New test case in `SQLMetricsSuite`. Closes apache#23287 from xuanyuanking/SPARK-26327-2.4. Authored-by: Yuanjian Li <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
What changes were proposed in this pull request?
Backport #23277 to branch 2.4 without the metrics renaming.
How was this patch tested?
New test case in
SQLMetricsSuite.