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 2021/11/13 04:13:19 UTC

[GitHub] [pulsar] michaeljmarshall opened a new pull request #12781: [Java Client] Let producer reconnect for state RegisteringSchema

michaeljmarshall opened a new pull request #12781:
URL: https://github.com/apache/pulsar/pull/12781


   ### Motivation
   
   In the Java Client, if a producer is in the `RegisteringSchema` state, it is a valid state for it to reconnect. Fix the `ConnectionHandler` to align with this behavior.
   
   ### Modifications
   
   * Update the `isValidStateForReconnection` method to return `true` for state `RegisteringSchema`.
   
   ### Verifying this change
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   ### Does this pull request potentially affect one of the following parts:
   
   This update does not contain breaking changes.
   
   ### Documentation
   
   - [x] `no-need-doc` 
   
   This is an implementation detail, so no need to update any documentation.
   
   


-- 
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] michaeljmarshall commented on pull request #12781: [Java Client] Let producer reconnect for state RegisteringSchema

Posted by GitBox <gi...@apache.org>.
michaeljmarshall commented on pull request #12781:
URL: https://github.com/apache/pulsar/pull/12781#issuecomment-967776703


   @congbobo184 - PTAL. This change aligns with your recent change to #12689.


-- 
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] 315157973 merged pull request #12781: [Java Client] Let producer reconnect for state RegisteringSchema

Posted by GitBox <gi...@apache.org>.
315157973 merged pull request #12781:
URL: https://github.com/apache/pulsar/pull/12781


   


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