Skip to content

Commit

Permalink
GEODE-10086: Removed StringBuilder variables (apache#7399)
Browse files Browse the repository at this point in the history
* Removed StringBuilder
* Using local string
  • Loading branch information
nabarunnag authored and mhansonp committed Mar 11, 2022
1 parent ccb6cfa commit d5fbfa5
Showing 1 changed file with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ public void cmdExecute(final @NotNull Message clientMessage,
final @NotNull ServerConnection serverConnection,
final @NotNull SecurityService securityService, long start)
throws IOException, InterruptedException {
StringBuilder errMessage = new StringBuilder();
CachedRegionHelper crHelper = serverConnection.getCachedRegionHelper();
CacheServerStats stats = serverConnection.getCacheServerStats();
serverConnection.setAsTrue(REQUIRES_RESPONSE);
Expand All @@ -73,10 +72,8 @@ public void cmdExecute(final @NotNull Message clientMessage,

if (regionName == null) {
logger.warn("The input region name for the %s request is null", "size");
errMessage
.append(String.format("The input region name for the %s request is null",
"size"));
writeErrorResponse(clientMessage, MessageType.SIZE_ERROR, errMessage.toString(),
writeErrorResponse(clientMessage, MessageType.SIZE_ERROR,
"The input region name for the size request is null",
serverConnection);
serverConnection.setAsTrue(RESPONDED);
return;
Expand Down

0 comments on commit d5fbfa5

Please sign in to comment.