Repository: kudu Updated Branches: refs/heads/master 8c8b16d4c -> f06a072e5
[java] Clean up a few usages of deprected methods Change-Id: I613fe910461dffc0c80a89046ad683592642f611 Reviewed-on: http://gerrit.cloudera.org:8080/9987 Reviewed-by: Grant Henke <granthe...@apache.org> Tested-by: Kudu Jenkins Project: http://git-wip-us.apache.org/repos/asf/kudu/repo Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/f06a072e Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/f06a072e Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/f06a072e Branch: refs/heads/master Commit: f06a072e5f512814f02afdd531567e1063e2daea Parents: 8c8b16d Author: Grant Henke <granthe...@apache.org> Authored: Tue Apr 10 10:19:50 2018 -0500 Committer: Grant Henke <granthe...@apache.org> Committed: Wed Apr 11 20:35:33 2018 +0000 ---------------------------------------------------------------------- .../kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java | 2 +- .../src/main/java/org/apache/kudu/util/DecimalUtil.java | 3 +-- .../src/main/java/org/apache/kudu/util/SecurityUtil.java | 2 +- .../kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java | 2 +- 4 files changed, 4 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kudu/blob/f06a072e/java/kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java ---------------------------------------------------------------------- diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java index f14a1d0..7c02ab4 100644 --- a/java/kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java +++ b/java/kudu-client/src/main/java/org/apache/kudu/client/IPCUtil.java @@ -71,7 +71,7 @@ public class IPCUtil { continue; } totalSize += m.getSerializedSize(); - totalSize += CodedOutputStream.computeRawVarint32Size(m.getSerializedSize()); + totalSize += CodedOutputStream.computeUInt32SizeNoTag(m.getSerializedSize()); } return totalSize; } http://git-wip-us.apache.org/repos/asf/kudu/blob/f06a072e/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java ---------------------------------------------------------------------- diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java index ae76481..19a470a 100644 --- a/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java +++ b/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java @@ -19,7 +19,6 @@ package org.apache.kudu.util; import java.math.BigDecimal; import java.math.BigInteger; -import java.math.MathContext; import java.math.RoundingMode; import com.google.common.base.Strings; @@ -123,7 +122,7 @@ public class DecimalUtil { public static BigDecimal coerce(BigDecimal val, int targetPrecision, int targetScale) { if (val.scale() != targetScale) { try { - val = val.setScale(targetScale, BigDecimal.ROUND_UNNECESSARY); + val = val.setScale(targetScale, RoundingMode.UNNECESSARY); } catch (ArithmeticException ex) { throw new IllegalArgumentException("Value scale " + val.scale() + " can't be coerced to target scale " + targetScale + ". "); http://git-wip-us.apache.org/repos/asf/kudu/blob/f06a072e/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java ---------------------------------------------------------------------- diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java index 35b61bf..e4697fd 100644 --- a/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java +++ b/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java @@ -155,7 +155,7 @@ public abstract class SecurityUtil { try { return MessageDigest.getInstance(digestAlg).digest(cert.getEncoded()); } catch (Exception e) { - throw Throwables.propagate(e); + throw new RuntimeException(e); } } http://git-wip-us.apache.org/repos/asf/kudu/blob/f06a072e/java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java ---------------------------------------------------------------------- diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java b/java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java index 0545b94..25ff5ed 100644 --- a/java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java +++ b/java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java @@ -82,7 +82,7 @@ public class FakeDNS { List<NameService> nameServices = (List<NameService>) field.get(null); nameServices.add(0, new NSImpl()); } catch (Exception e) { - throw Throwables.propagate(e); + throw new RuntimeException(e); } installed = true; }