[geode] branch develop updated: GEODE-7864: Print contents of arrays correctly. (#4879)

[nnag]

This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 2cf9925  GEODE-7864: Print contents of arrays correctly. (#4879)
2cf9925 is described below

commit 2cf9925047ab60bf33979b718222674d00c6b3f7
Author: Nabarun Nag <nabarun...@users.noreply.github.com>
AuthorDate: Mon Mar 30 11:16:19 2020 -0700

    GEODE-7864: Print contents of arrays correctly. (#4879)
---
 .../internal/cache/tier/sockets/ClientDataSerializerMessage.java    | 2 +-
 .../internal/cache/tier/sockets/ClientInstantiatorMessage.java      | 2 +-
 .../org/apache/geode/internal/cache/functions/TestFunction.java     | 6 ++++--
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientDataSerializerMessage.java
 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientDataSerializerMessage.java
index 71a69d2..2936903 100644
--- 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientDataSerializerMessage.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientDataSerializerMessage.java
@@ -229,7 +229,7 @@ public class ClientDataSerializerMessage extends 
ClientUpdateMessageImpl {
   public String toString() {
     StringBuilder buffer = new StringBuilder();
     buffer.append("ClientDataSerializerMessage[value=")
-        .append(Arrays.toString(this.serializedDataSerializer))
+        .append(Arrays.deepToString(this.serializedDataSerializer))
         .append(";memberId=")
         
.append(getMembershipId()).append(";eventId=").append(getEventId()).append("]");
     return buffer.toString();
diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
index 988cb30..4d791b1 100755
--- 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
@@ -204,7 +204,7 @@ public class ClientInstantiatorMessage extends 
ClientUpdateMessageImpl {
   public String toString() {
     StringBuffer buffer = new StringBuffer();
     buffer.append("ClientInstantiatorMessage[value=")
-        .append(Arrays.toString(this.serializedInstantiators))
+        .append(Arrays.deepToString(this.serializedInstantiators))
         .append(";memberId=")
         .append(getMembershipId()).append(";eventId=").append(getEventId())
         .append("]");
diff --git 
a/geode-dunit/src/main/java/org/apache/geode/internal/cache/functions/TestFunction.java
 
b/geode-dunit/src/main/java/org/apache/geode/internal/cache/functions/TestFunction.java
index 77de9ed..6266d86 100755
--- 
a/geode-dunit/src/main/java/org/apache/geode/internal/cache/functions/TestFunction.java
+++ 
b/geode-dunit/src/main/java/org/apache/geode/internal/cache/functions/TestFunction.java
@@ -18,6 +18,7 @@ import java.io.DataInput;
 import java.io.DataOutput;
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
@@ -212,10 +213,11 @@ public class TestFunction<T> implements Function<T>, 
Declarable2, DataSerializab
     RegionFunctionContextImpl rfc = (RegionFunctionContextImpl) context;
     PartitionedRegion pr = (PartitionedRegion) rfc.getDataSet();
     int[] bucketIDs = rfc.getLocalBucketArray(pr);
-    pr.getGemFireCache().getLogger().fine("LOCAL BUCKETSET =" + bucketIDs);
+    pr.getGemFireCache().getLogger().fine("LOCAL BUCKETSET =" + 
Arrays.toString(bucketIDs));
     ResultSender<Integer> rs = context.<Integer>getResultSender();
     if (!pr.getDataStore().areAllBucketsHosted(bucketIDs)) {
-      throw new AssertionError("bucket IDs =" + bucketIDs + " not all hosted 
locally");
+      throw new AssertionError(
+          "bucket IDs =" + Arrays.toString(bucketIDs) + " not all hosted 
locally");
     } else {
       for (int i = 1; i < bucketIDs[0]; ++i) {
         rs.sendResult(bucketIDs[i]);