Skip to content

Commit

Permalink
[Java] Avoid allocation for empty cluster members list and increase A…
Browse files Browse the repository at this point in the history
…eron version.
  • Loading branch information
mjpt777 committed Dec 16, 2019
1 parent 3583fe1 commit 297a260
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -566,6 +566,11 @@ public static ClusterMember parseEndpoints(final int id, final String endpointsD
*/
public static String encodeAsString(final ClusterMember[] clusterMembers)
{
if (0 == clusterMembers.length)
{
return "";
}

final StringBuilder builder = new StringBuilder();

for (int i = 0, length = clusterMembers.length; i < length; i++)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -778,9 +778,9 @@ public void onRemoveMember(final long correlationId, final int memberId, final b
}
}

public void onClusterMembersQuery(final long correlationId, final boolean returnExtended)
public void onClusterMembersQuery(final long correlationId, final boolean isExtendedRequest)
{
if (returnExtended)
if (isExtendedRequest)
{
serviceProxy.clusterMembersExtendedResponse(
correlationId, timeNs, leaderMember.id(), memberId, clusterMembers, passiveMembers);
Expand Down
2 changes: 1 addition & 1 deletion version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.24.1-SNAPSHOT
1.25.0-SNAPSHOT

0 comments on commit 297a260

Please sign in to comment.