merlimat commented on a change in pull request #309:
URL: https://github.com/apache/pulsar-client-go/pull/309#discussion_r450984529
##########
File path: pulsar/internal/connection.go
##########
@@ -288,7 +288,7 @@ func (c *connection) doHandshake() bool {
if cmd.Connected == nil {
c.log.Warnf("Failed to perform initial handshake - Expecting
'Connected' cmd, got '%s'",
- cmd.Type)
+ cmd.Error.GetMessage())
Review comment:
Please change the original message since with this change it wouldn't
make sense anymore:
```
c.log.Warnf("Failed to establish connection with broker: '%s'",
```
----------------------------------------------------------------
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:
[email protected]
