codelipenghui commented on issue #9803: URL: https://github.com/apache/pulsar/issues/9803#issuecomment-790588032
@james-bright-helix I think the problem is here https://github.com/apache/pulsar/blob/2fd878a8f4bfdd5cffa7fa6450714ec1ad25f514/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java#L408, since when the consumer receives a message, the properties of the MessageImpl is not null, we update the message builder then use msg.getProperties() to get the properties will return the old properties. The reconsumeLater method only copies the properties by msg.getProperties(), so I think this is the root cause why the properties missing. Seems we can fix the msg.getProperties() method, if the properties of the message metadata changed, we should copy the properties from the metadata to the MessageImpl. ---------------------------------------------------------------- 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: [email protected]
