Technoboy- commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1063986821


   > > > @codelipenghui
   > > > The delays is placed in the MessageMetadata. This feature will make 
the broker additional a deserialization MessageMetaData when receiving the 
producer message. Can we accept the performance impact?
   > > 
   > > 
   > > Or, We need add new field `hasDeliver` to `CommandSend`, This can avoid 
deserialization the data of `MessageMetadata`. This is an API protocol change, 
We need to discuss first.
   > > @codelipenghui @Technoboy- PTAL
   > 
   > I will help to check first and then decide to send out a discussion email.
   
   Yes, you're right. If add this field in the `CommandSend`, it is duplicated 
with `MsgMetadata`, and deserialization is much heavier, so after discussion, 
we decide to close this issue now. 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to