[GitHub] [pulsar-client-go] merlimat commented on a change in pull request #299: Support producer name for Message

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

merlimat commented on a change in pull request #299:
URL: https://github.com/apache/pulsar-client-go/pull/299#discussion_r445954733



##########
File path: pulsar/message.go
##########
@@ -64,6 +64,9 @@ type Message interface {
 	// Topic get the topic from which this message originated from
 	Topic() string
 
+	// ProducerName will get producer name from broker

Review comment:
       Saying "from broker" can be misleading since we're not fetching it from the broker, rather the name comes attached to the message metadata.
   
   In Java client doc we have: 
   ```
   Get the producer name who produced this message.
   ```
   
   ```suggestion
   	// ProducerName returns the name of the producer that has published the message
   ```




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