You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by GitBox <gi...@apache.org> on 2020/08/07 15:59:56 UTC

[GitHub] [samza] sborya opened a new pull request #1410: make MonitorPartition thread a daemon

sborya opened a new pull request #1410:
URL: https://github.com/apache/samza/pull/1410


   The monitor needs to shutdown whenever the applications shuts down regardless of its state.


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

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



[GitHub] [samza] mynameborat commented on pull request #1410: make MonitorPartition thread a daemon

Posted by GitBox <gi...@apache.org>.
mynameborat commented on pull request #1410:
URL: https://github.com/apache/samza/pull/1410#issuecomment-670596782


   Thanks for the fix!
   Can you update the PR description and title to follow - https://cwiki.apache.org/confluence/display/SAMZA/SEP-25%3A+PR+Title+And+Description+Guidelines before merging the 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.

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



[GitHub] [samza] sborya merged pull request #1410: make MonitorPartition thread a daemon

Posted by GitBox <gi...@apache.org>.
sborya merged pull request #1410:
URL: https://github.com/apache/samza/pull/1410


   


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

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



[GitHub] [samza] sborya commented on pull request #1410: make MonitorPartition thread a daemon

Posted by GitBox <gi...@apache.org>.
sborya commented on pull request #1410:
URL: https://github.com/apache/samza/pull/1410#issuecomment-670652787


   > Thanks for the fix!
   > Can you update the PR description and title to follow - https://cwiki.apache.org/confluence/display/SAMZA/SEP-25%3A+PR+Title+And+Description+Guidelines before merging the change.
   
   Sure. Will do.


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

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