This is an automated email from the ASF dual-hosted git repository. ctubbsii pushed a commit to branch pr/1412 in repository https://gitbox.apache.org/repos/asf/accumulo.git
commit 51e4948e0546870911f90aabdf80174d321d4c9a Author: Christopher Tubbs <[email protected]> AuthorDate: Thu Nov 7 18:42:13 2019 -0500 PR feedback --- core/src/test/java/org/apache/accumulo/core/data/ValueTest.java | 2 +- test/src/main/java/org/apache/accumulo/test/util/SlowOps.java | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/core/src/test/java/org/apache/accumulo/core/data/ValueTest.java b/core/src/test/java/org/apache/accumulo/core/data/ValueTest.java index f7d3ec2..a8b6ba4 100644 --- a/core/src/test/java/org/apache/accumulo/core/data/ValueTest.java +++ b/core/src/test/java/org/apache/accumulo/core/data/ValueTest.java @@ -194,7 +194,7 @@ public class ValueTest { @Test public void testString() { Value v1 = new Value("abc"); - Value v2 = new Value("abc"); + Value v2 = new Value("abc".getBytes(UTF_8)); assertEquals(v2, v1); } diff --git a/test/src/main/java/org/apache/accumulo/test/util/SlowOps.java b/test/src/main/java/org/apache/accumulo/test/util/SlowOps.java index 5e28a06..177f544 100644 --- a/test/src/main/java/org/apache/accumulo/test/util/SlowOps.java +++ b/test/src/main/java/org/apache/accumulo/test/util/SlowOps.java @@ -16,8 +16,6 @@ */ package org.apache.accumulo.test.util; -import static java.nio.charset.StandardCharsets.UTF_8; - import java.util.ArrayList; import java.util.List; import java.util.Map; @@ -133,8 +131,7 @@ public class SlowOps { // populate for (int i = 0; i < NUM_DATA_ROWS; i++) { Mutation m = new Mutation(new Text(String.format("%05d", i))); - m.put(new Text("col" + ((i % 3) + 1)), new Text("qual"), - new Value("junk".getBytes(UTF_8))); + m.put(new Text("col" + ((i % 3) + 1)), new Text("qual"), new Value("junk")); bw.addMutation(m); } }
