Ariel Weisberg commented on CASSANDRA-14225:

Did you want me as the reviewer or assignee?

{{FBUtilities.getBroadcastNativeAddressAndPort()}} is for the port used by the 
CQL protocol. Maybe we should javadoc that. You want 
{{FBUtilities.getBroadcastAddressAndPort()}} for the storage port. There is a 
missing newline between the new import and the start of the class comments.

In RepairRunnable the diff introduces a tab when constructing the 

Otherwise +1.

> fix comparison of address and port for repair and messages
> ----------------------------------------------------------
>                 Key: CASSANDRA-14225
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-14225
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Dave Brosius
>            Assignee: Ariel Weisberg
>            Priority: Trivial
>             Fix For: 4.0
>         Attachments: 14225.txt
> compare both host and port, not just host

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to