[GitHub] [pulsar] sijie commented on issue #5043: [zookeeper] bump zookeeper to version 3.5.5

[GitBox <gi...@apache.org>]

sijie commented on issue #5043: [zookeeper] bump zookeeper to version 3.5.5
URL: https://github.com/apache/pulsar/pull/5043#issuecomment-525586878
 
 
   >  I had thought it might make sense to do a follow up for docs. 
   
   Sorry that I missed your comment before. But since this is related to upgrade guide, can you please create a github issue and link the issue here? We have to mark that issue as a blocker for releasing 2.5.0.
   
   >  I assume it may make sense to keep them around for some period of time for those who don't use embedded zookeeper and are sticking around 3.4?
   
   I think the hack was done using AspectJ to inject `deserializeTxn` logic for handling a mixed transaction types between 3.4.x and 3.5.x. Hence I don't feel comfortable to have this hack logic applied to 3.5.x especially it is about deserializing transaction from zookeeper journals.
   
   

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


With regards,
Apache Git Services