wolfstudy commented on a change in pull request #648:
URL: https://github.com/apache/pulsar-client-go/pull/648#discussion_r737383331
##########
File path: pulsar/producer_partition.go
##########
@@ -60,13 +60,16 @@ var (
)
var errTopicNotFount = "TopicNotFound"
+var errMetadata = "MetadataError"
+
type partitionProducer struct {
state ua.Int32
client *client
topic string
log log.Logger
cnx internal.Connection
+ err error
Review comment:
hello @cckellogg , It seems that some of our Error messages are not
fully exposed, such as sending failures after reaching the Backlog threshold.
It seems that @yorkhellen is encapsulating their business based on the Go SDK,
and may need to process specific business logic based on these specific errors.
It seems that it is OK to expose the error field to the user here. This way the
business can be more flexible in error handling.
--
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]
