Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-05-07 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-2097692402 > Please let us know if there is any change on this PR. In the previous tests, I found a case `hang`. I'll rebase it again -- This is an automated message from the Apache

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-05-06 Thread via GitHub
dongjoon-hyun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-2096760979 Please let us know if there is any change on this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-18 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-2005598716 There are still some problems. After I finish this PR https://github.com/apache/spark/pull/45551, I will carefully analyze it. -- This is an automated message from the Apache

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-18 Thread via GitHub
LuciferYang commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-2003320274 ready to go? -- 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

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-12 Thread via GitHub
dongjoon-hyun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1991786269 Thank you for making it sure, @panbingkun .  -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-12 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1991472542 GA runs successfully, as recorded below: -- 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

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-12 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1990853200 > @panbingkun We can enable the debug logs of the RemoteSparkSession subclass to facilitate troubleshooting. Done. -- This is an automated message from the Apache Git

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-12 Thread via GitHub
LuciferYang commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1990791157 @panbingkun We can enable the debug logs of the RemoteSparkSession subclass to facilitate troubleshooting. -- This is an automated message from the Apache Git Service. To

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-11 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1990682229 @dongjoon-hyun To be more `reliable`, I will convert this pr from `draft` to `review` until GA can run successfully. I have tried the test `ClientStreamingQuerySuite ` several

Re: [PR] [WIP][SPARK-47194][BUILD] Upgrade log4j to 2.23.1 [spark]

2024-03-11 Thread via GitHub
panbingkun commented on PR #45326: URL: https://github.com/apache/spark/pull/45326#issuecomment-1987848933 @LuciferYang @dongjoon-hyun The version `2.23.1` of log4j2 has been released, which has resolved the issue with the `StatusLogger` in the version `1.13.0`, as follows: