Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-11 Thread via GitHub
ferenc-csaky merged PR #25463: URL: https://github.com/apache/flink/pull/25463 -- 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 unsubscribe, e-mail: issues-unsubscr...@flink.

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-10 Thread via GitHub
r-sidd commented on PR #25463: URL: https://github.com/apache/flink/pull/25463#issuecomment-2405549278 > LGTM, will merge by Friday EOD (CEST) if no objections until then. Sure 🫡 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-09 Thread via GitHub
r-sidd commented on PR #25463: URL: https://github.com/apache/flink/pull/25463#issuecomment-2404069277 @ferenc-csaky I've pushed the changes now -- 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 th

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-09 Thread via GitHub
r-sidd commented on PR #25463: URL: https://github.com/apache/flink/pull/25463#issuecomment-2404062534 > Can you pls update the version in the notice file as well, like here: #18228 > > (The documentation do not contain it anymore.) Sure thing, will do now -- This is an auto

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-09 Thread via GitHub
Juoelenis commented on PR #25463: URL: https://github.com/apache/flink/pull/25463#issuecomment-2404028171 I aproovd -- 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 unsub

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-09 Thread via GitHub
ferenc-csaky commented on code in PR #25463: URL: https://github.com/apache/flink/pull/25463#discussion_r1794652460 ## pom.xml: ## @@ -127,7 +127,7 @@ under the License. true 11 1.7.36 - 2.17.1 + 2.23.1

Re: [PR] [FLINK-36440] Bump log4j from 2.17.1 to 2.24.1 [flink]

2024-10-09 Thread via GitHub
r-sidd commented on code in PR #25463: URL: https://github.com/apache/flink/pull/25463#discussion_r1794572630 ## pom.xml: ## @@ -127,7 +127,7 @@ under the License. true 11 1.7.36 - 2.17.1 + 2.23.1 Revie