Re: [PR] [SPARK-51374][CORE] Switch to Using java.util.Map in Logging APIs [spark]

2025-03-11 Thread via GitHub
panbingkun commented on PR #50138: URL: https://github.com/apache/spark/pull/50138#issuecomment-2716655284 LGTM late. -- 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 to the specific comment. To uns

Re: [PR] [SPARK-51374][CORE] Switch to Using java.util.Map in Logging APIs [spark]

2025-03-03 Thread via GitHub
gengliangwang closed pull request #50138: [SPARK-51374][CORE] Switch to Using java.util.Map in Logging APIs URL: https://github.com/apache/spark/pull/50138 -- 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

Re: [PR] [SPARK-51374][CORE] Switch to Using java.util.Map in Logging APIs [spark]

2025-03-03 Thread via GitHub
gengliangwang commented on PR #50138: URL: https://github.com/apache/spark/pull/50138#issuecomment-2695729807 @cnauroth @dongjoon-hyun Thanks for the review. The test failure is not relevant. I am merging this one to master and branch-4.0 -- This is an automated message from the Apac

[PR] [SPARK-51374][CORE] Switch to Using java.util.Map in Logging APIs [spark]

2025-03-03 Thread via GitHub
gengliangwang opened a new pull request, #50138: URL: https://github.com/apache/spark/pull/50138 ### What changes were proposed in this pull request? Update the internal logging APIs to use `java.util.Map` instead of the concrete `java.util.HashMap`. ### Why are the cha