You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2019/11/07 23:42:33 UTC

[accumulo] 01/01: PR feedback

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 <ct...@apache.org>
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);
         }
       }