rdhabalia commented on issue #1462: Delayed acks impl URL: https://github.com/apache/incubator-pulsar/pull/1462#issuecomment-403256971 @merlimat this is not a backward compatible change with old broker (version < 2.0) because, old brokers don't understand group-ack command so, combination of new-client (>=2.0) and old-broker will not work and it can create ack-holes for the client. also `CommandAck` schema is also changed ``` Before: required MessageIdData message_id = 3; After: repeated MessageIdData message_id = 3; ``` I think we were making sure that we don't break backward compatibility so, If client uses new version and if it requires to rollback broker version then it can not create an issue here. Any thoughts to fix this issue?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
