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/07/12 22:02:27 UTC
[accumulo] branch master updated: Remove some build warnings
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 <ct...@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>