You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/11/25 02:38:06 UTC

[GitHub] [flink] XComp opened a new pull request, #21394: [FLINK-26037][coordination] TMRunner waits for metric registry shutdown

XComp opened a new pull request, #21394:
URL: https://github.com/apache/flink/pull/21394

   1.15 backport PR of PR #21346
   


-- 
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

[GitHub] [flink] XComp commented on pull request #21394: [FLINK-26037][coordination] TMRunner waits for metric registry shutdown

Posted by GitBox <gi...@apache.org>.
XComp commented on PR #21394:
URL: https://github.com/apache/flink/pull/21394#issuecomment-1326964142

   Considering that `MetricRegistryImpl#shutdown` was already implemented in an asynchronous manner before FLINK-29488 was merged, I tend to believe that it might not be the cause of FLINK-26037 and related issues. Otherwise, we would have seen this error already earlier. :thinking: 


-- 
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

[GitHub] [flink] flinkbot commented on pull request #21394: [FLINK-26037][coordination] TMRunner waits for metric registry shutdown

Posted by GitBox <gi...@apache.org>.
flinkbot commented on PR #21394:
URL: https://github.com/apache/flink/pull/21394#issuecomment-1326963911

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "bbccf13c31087012d1d0c8adfea5cd78beb457e1",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "bbccf13c31087012d1d0c8adfea5cd78beb457e1",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * bbccf13c31087012d1d0c8adfea5cd78beb457e1 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

[GitHub] [flink] zentol commented on pull request #21394: [BP-1.15][FLINK-26037][coordination] TMRunner waits for metric registry shutdown

Posted by GitBox <gi...@apache.org>.
zentol commented on PR #21394:
URL: https://github.com/apache/flink/pull/21394#issuecomment-1327225053

   > Otherwise, we would have seen this error already earlier. 
   
   Not necessarily though; you just need 1 step of the shutdown procedure being slow enough.


-- 
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

[GitHub] [flink] XComp commented on pull request #21394: [BP-1.15][FLINK-26037][coordination] TMRunner waits for metric registry shutdown

Posted by GitBox <gi...@apache.org>.
XComp commented on PR #21394:
URL: https://github.com/apache/flink/pull/21394#issuecomment-1326973692

   force push was accidental with no actual change.


-- 
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

[GitHub] [flink] XComp merged pull request #21394: [BP-1.15][FLINK-26037][coordination] TMRunner waits for metric registry shutdown

Posted by GitBox <gi...@apache.org>.
XComp merged PR #21394:
URL: https://github.com/apache/flink/pull/21394


-- 
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