You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2020/01/29 17:58:44 UTC
[kudu] 02/02: [java] Use Java built-in Base64 to generate
randomString
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit c80080d365aa478a951cbbd76475a52a0767ac82
Author: Grant Henke <gr...@apache.org>
AuthorDate: Wed Jan 29 09:35:05 2020 -0600
[java] Use Java built-in Base64 to generate randomString
Currently we use DatatypeConverter to generate random Base64
Strings in DataGenerator.java. This patch changes to use the
Java built-in Base64 class instead.
Change-Id: I99d511bb8c161159b3ee477460246bc368cc2979
Reviewed-on: http://gerrit.cloudera.org:8080/15127
Tested-by: Grant Henke <gr...@apache.org>
Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
.../kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java b/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
index 095e3c9..0e281bc 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
@@ -19,9 +19,9 @@ package org.apache.kudu.util;
import java.math.BigDecimal;
import java.math.BigInteger;
+import java.util.Base64;
import java.util.List;
import java.util.Random;
-import javax.xml.bind.DatatypeConverter;
import com.google.common.base.Preconditions;
import org.apache.yetus.audience.InterfaceAudience;
@@ -148,7 +148,7 @@ public class DataGenerator {
public static String randomString(int length, Random random) {
byte[] bytes = new byte[length];
random.nextBytes(bytes);
- return DatatypeConverter.printBase64Binary(bytes);
+ return Base64.getEncoder().encodeToString(bytes);
}
/**