apply review comments

elastic · Jul 5, 2016 · 7bb36b9 · 7bb36b9
1 parent 10164e7
commit 7bb36b9
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/core/src/main/java/org/elasticsearch/common/bytes/CompositeBytesReference.java b/core/src/main/java/org/elasticsearch/common/bytes/CompositeBytesReference.java
@@ -112,7 +112,7 @@ public BytesRef toBytesRef() {
                 builder.append(spare);
             }
         } catch (IOException ex) {
-            throw new AssertionError("won't happen", ex);
+            throw new AssertionError("won't happen", ex); // this is really an error since we don't do IO in our bytesreferences
         }
         return builder.toBytesRef();
     }

diff --git a/core/src/test/java/org/elasticsearch/cluster/MinimumMasterNodesIT.java b/core/src/test/java/org/elasticsearch/cluster/MinimumMasterNodesIT.java
@@ -62,6 +62,7 @@
 
 @ClusterScope(scope = Scope.TEST, numDataNodes = 0)
 @ESIntegTestCase.SuppressLocalMode
+@TestLogging("_root:DEBUG,cluster.service:TRACE,discovery.zen:TRACE")
 public class MinimumMasterNodesIT extends ESIntegTestCase {
 
     @Override

diff --git a/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java b/core/src/test/java/org/elasticsearch/common/bytes/AbstractBytesReferenceTestCase.java
@@ -147,10 +147,7 @@ public void testStreamInputMarkAndReset() throws IOException {
             }
             assertEquals(si.readByte(), wrap.readByte());
             assertEquals(si.available(), wrap.available());
-
         }
-
-
     }
 
     public void testStreamInputBulkReadWithOffset() throws IOException {