[GitHub] [pulsar] BewareMyPower commented on issue #9176: Python Client Producer Synchronous Send Void Return Value

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

BewareMyPower commented on issue #9176:
URL: https://github.com/apache/pulsar/issues/9176#issuecomment-758340078


   Yes, it's also a problem of C++ client that synchronous send can't retrieve the message id. It's an issue of the initial API design.
   
   The fix may change the API, I'm not sure if it's proper. A simple fix that keeps the compatibility may be adding a overload method to `Producer::send`:
   
   ```c++
       Result send(const Message& msg);  // the original API
   
       Result send(const Message& msg, MessageId& msgId);  // the new API
   ```


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