You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "zentol (via GitHub)" <gi...@apache.org> on 2024/02/22 17:14:49 UTC

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

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 class at the same time because during class initialization they hold a lock.
   
   Break this dependency by moving `hasJsonCreatorAnnotation` to `ExecNodeMetadataUtil`.


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

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

Posted by "zentol (via GitHub)" <gi...@apache.org>.
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

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

Posted by "flinkbot (via GitHub)" <gi...@apache.org>.
flinkbot commented on PR #24371:
URL: https://github.com/apache/flink/pull/24371#issuecomment-1959914821

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "eb6b6ed2c3c2144bfea93ec201f22e1dd7e3bd25",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "eb6b6ed2c3c2144bfea93ec201f22e1dd7e3bd25",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * eb6b6ed2c3c2144bfea93ec201f22e1dd7e3bd25 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run azure` re-run the last Azure build
   </details>


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

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

Posted by "snuyanzin (via GitHub)" <gi...@apache.org>.
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 and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

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

Posted by "zentol (via GitHub)" <gi...@apache.org>.
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 the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org