diff --git a/storage-api/src/java/org/apache/hadoop/hive/common/io/encoded/EncodedColumnBatch.java b/storage-api/src/java/org/apache/hadoop/hive/common/io/encoded/EncodedColumnBatch.java index 29a3b0f2f4..db3a3dc26b 100644 --- a/storage-api/src/java/org/apache/hadoop/hive/common/io/encoded/EncodedColumnBatch.java +++ b/storage-api/src/java/org/apache/hadoop/hive/common/io/encoded/EncodedColumnBatch.java @@ -19,6 +19,7 @@ package org.apache.hadoop.hive.common.io.encoded; import java.util.Arrays; +import java.util.Iterator; import java.util.List; import java.util.concurrent.atomic.AtomicInteger; @@ -78,13 +79,21 @@ public void setIndexBaseOffset(int indexBaseOffset) { @Override public String toString() { - String bufStr = ""; + StringBuilder sb = new StringBuilder(); if (cacheBuffers != null) { - for (MemoryBuffer mb : cacheBuffers) { - bufStr += mb.getClass().getSimpleName() + " with " + mb.getByteBufferRaw().remaining() + " bytes, "; + Iterator iter = cacheBuffers.iterator(); + while (iter.hasNext()) { + MemoryBuffer mb = iter.next(); + sb.append(mb.getClass().getSimpleName()); + sb.append(" with "); + sb.append(mb.getByteBufferRaw().remaining()); + sb.append(" bytes"); + if (iter.hasNext()) { + sb.append(", "); + } } } - return "ColumnStreamData [cacheBuffers=[" + bufStr + return "ColumnStreamData [cacheBuffers=[" + sb + "], indexBaseOffset=" + indexBaseOffset + "]"; }