You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/06/03 09:56:47 UTC

[GitHub] [pulsar] lhotari opened a new pull request, #15922: [Broker] Close transactionBufferClient before closing the internal Pulsar client

lhotari opened a new pull request, #15922:
URL: https://github.com/apache/pulsar/pull/15922

   ### Motivation
   
   - `transactionBufferClient` uses the internal Pulsar client and therefore it should be closed before the internal Pulsar client.
   
   ### Modifications
   
   - close `transactionsBufferClient` before the internal Pulsar client in `PulsarService`
   
   ### Additional context
   
   I noticed the shutdown order inconsistency while investigating the flaky test failure #15920 .


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] merlimat merged pull request #15922: [Broker] Close transactionBufferClient before closing the internal Pulsar client

Posted by GitBox <gi...@apache.org>.

merlimat merged PR #15922:
URL: https://github.com/apache/pulsar/pull/15922


-- 
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: commits-unsubscribe@pulsar.apache.org

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