Re: [PR] [FLINK-34496] Break circular dependency in static initializatio [flink]

2024-02-23 Thread via GitHub
zentol merged PR #24371: URL: https://github.com/apache/flink/pull/24371 -- 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.apache

Re: [PR] [FLINK-34496] Break circular dependency in static initializatio [flink]

2024-02-22 Thread via GitHub
zentol commented on PR #24371: URL: https://github.com/apache/flink/pull/24371#issuecomment-1960125803 > A tiny nitpick: it seems there is a misprint in commit message/PR name Will fix that during the merge. -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] [FLINK-34496] Break circular dependency in static initializatio [flink]

2024-02-22 Thread via GitHub
snuyanzin commented on PR #24371: URL: https://github.com/apache/flink/pull/24371#issuecomment-1959947072 A tiny nitpick: it seems there is a misprint in commit message/PR name -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [PR] [FLINK-34496] Break circular dependency in static initializatio [flink]

2024-02-22 Thread via GitHub
flinkbot commented on PR #24371: URL: https://github.com/apache/flink/pull/24371#issuecomment-1959914821 ## CI report: * eb6b6ed2c3c2144bfea93ec201f22e1dd7e3bd25 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34496] Break circular dependency in static initializatio [flink]

2024-02-22 Thread via GitHub
zentol opened a new pull request, #24371: URL: https://github.com/apache/flink/pull/24371 `ExecNodeMetadataUtil` and `JsonSerdeUtil` have a circular dependency in their static initialization, which can cause a classloading lockup when 2 threads are running the class initialization of each c