accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [accumulo] branch master updated: Remove some build warnings
Date Fri, 12 Jul 2019 22:02:27 GMT
This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/master by this push:
     new 35a5c7d  Remove some build warnings
35a5c7d is described below

commit 35a5c7d45a1a4f590e92dee0e61cfef7f1640df2
Author: Christopher Tubbs <ctubbsii@apache.org>
AuthorDate: Fri Jul 12 18:02:12 2019 -0400

    Remove some build warnings
    
    * Avoid deprecated NumberUtils.isNumber
    * Remove duplicate dependency declaration
---
 .../org/apache/accumulo/core/iterators/FirstEntryInRowIterator.java   | 2 +-
 test/pom.xml                                                          | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/iterators/FirstEntryInRowIterator.java
b/core/src/main/java/org/apache/accumulo/core/iterators/FirstEntryInRowIterator.java
index 77d34f1..f87d0cd 100644
--- a/core/src/main/java/org/apache/accumulo/core/iterators/FirstEntryInRowIterator.java
+++ b/core/src/main/java/org/apache/accumulo/core/iterators/FirstEntryInRowIterator.java
@@ -147,7 +147,7 @@ public class FirstEntryInRowIterator extends SkippingIterator implements
OptionD
   @Override
   public boolean validateOptions(Map<String,String> options) {
     String o = options.get(NUM_SCANS_STRING_NAME);
-    if (o != null && !NumberUtils.isNumber(o))
+    if (o != null && !NumberUtils.isParsable(o))
       throw new IllegalArgumentException(
           "bad integer " + NUM_SCANS_STRING_NAME + ":" + options.get(NUM_SCANS_STRING_NAME));
     return true;
diff --git a/test/pom.xml b/test/pom.xml
index 63fec28..5b8f39f 100644
--- a/test/pom.xml
+++ b/test/pom.xml
@@ -124,10 +124,6 @@
       <artifactId>commons-lang3</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.apache.commons</groupId>
-      <artifactId>commons-lang3</artifactId>
-    </dependency>
-    <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-client-api</artifactId>
     </dependency>


Mime
View raw message