[GitHub] [spark] dongjoon-hyun commented on pull request #42935: [SPARK-45173][UI] Remove some unnecessary sourceMapping files in UI

2023-09-15 Thread via GitHub
dongjoon-hyun commented on PR #42935: URL: https://github.com/apache/spark/pull/42935#issuecomment-1722101732 Merged to master for Apache Spark 4.0.0. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [spark] dongjoon-hyun commented on pull request #42935: [SPARK-45173][UI] Remove some unnecessary sourceMapping files in UI

2023-09-15 Thread via GitHub
dongjoon-hyun commented on PR #42935: URL: https://github.com/apache/spark/pull/42935#issuecomment-1722101587 Thank you for the confirmation and updating the PR description. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] dongjoon-hyun commented on pull request #42935: [SPARK-45173][UI] Remove some unnecessary sourceMapping files in UI

2023-09-15 Thread via GitHub
dongjoon-hyun commented on PR #42935: URL: https://github.com/apache/spark/pull/42935#issuecomment-1721244501 If possible, please elaborate a little more in the PR description, @yaooqinn . :) -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [spark] dongjoon-hyun commented on pull request #42935: [SPARK-45173][UI] Remove some unnecessary sourceMapping files in UI

2023-09-14 Thread via GitHub
dongjoon-hyun commented on PR #42935: URL: https://github.com/apache/spark/pull/42935#issuecomment-1720620926 Ah, I missed that it's one line change. Got it. Thanks. ![Screenshot 2023-09-14 at 9 39 40  PM](https://github.com/apache/spark/assets/9700541/554dd6c0-23ce-4399-b413-e70486108

[GitHub] [spark] dongjoon-hyun commented on pull request #42935: [SPARK-45173][UI] Remove some unnecessary sourceMapping files in UI

2023-09-14 Thread via GitHub
dongjoon-hyun commented on PR #42935: URL: https://github.com/apache/spark/pull/42935#issuecomment-1720520890 If this is really unused, I believe - We need to remove `UIUtils.commonHeaderNodes` first. - Also need to remove from RAT file https://github.com/apache/spark/blob/91ccc0f