You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/07/06 03:27:24 UTC

[GitHub] [pulsar] freeznet commented on issue #11193: Document that the message key is the partition key in the Protobuf message

freeznet commented on issue #11193:
URL: https://github.com/apache/pulsar/issues/11193#issuecomment-874431813


   @Anonymitaet agree with @Geal, it is quite confusing about the message key and the partition key. From the interface of `Message`, we have `getKey()` but the implementation of `getKey()` returns the partition key.
   https://github.com/apache/pulsar/blob/fe7cf676e59e08b34b388fce3a190133d390d61c/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java#L561-L567
   According to @Geal's suggestion, a clarified doc should be added to let users aware that for the implementation of `Message`, we use the partition key as the message key.
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org