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);
   }
 
   /**