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:32 UTC
[accumulo] branch pr/1412 created (now 51e4948)
This is an automated email from the ASF dual-hosted git repository.
ctubbsii pushed a change to branch pr/1412
in repository https://gitbox.apache.org/repos/asf/accumulo.git.
at 51e4948 PR feedback
This branch includes the following new commits:
new 51e4948 PR feedback
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
[accumulo] 01/01: PR feedback
Posted by ct...@apache.org.
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);
}
}