You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2014/06/13 04:36:27 UTC
[5/9] git commit: ACCUMULO-2663 Fix some compilation issues from
guava deps
ACCUMULO-2663 Fix some compilation issues from guava deps
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/3d469eb8
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/3d469eb8
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/3d469eb8
Branch: refs/heads/1.6.1-SNAPSHOT
Commit: 3d469eb853a56ea5e1d0e15c049eb0284d2046c3
Parents: 5959127
Author: Josh Elser <el...@apache.org>
Authored: Thu Jun 12 22:28:56 2014 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Thu Jun 12 22:28:56 2014 -0400
----------------------------------------------------------------------
.../java/org/apache/accumulo/core/iterators/Combiner.java | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/accumulo/blob/3d469eb8/core/src/main/java/org/apache/accumulo/core/iterators/Combiner.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/accumulo/core/iterators/Combiner.java b/core/src/main/java/org/apache/accumulo/core/iterators/Combiner.java
index dd215cc..8ec22fe 100644
--- a/core/src/main/java/org/apache/accumulo/core/iterators/Combiner.java
+++ b/core/src/main/java/org/apache/accumulo/core/iterators/Combiner.java
@@ -16,15 +16,12 @@
*/
package org.apache.accumulo.core.iterators;
-import com.google.common.base.Splitter;
import java.io.IOException;
-import java.util.Arrays;
import java.util.Collection;
import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.NoSuchElementException;
-import java.util.StringTokenizer;
import org.apache.accumulo.core.client.IteratorSetting;
import org.apache.accumulo.core.client.IteratorSetting.Column;
@@ -38,6 +35,9 @@ import org.apache.accumulo.core.iterators.conf.ColumnSet;
import org.apache.hadoop.io.Text;
import org.apache.log4j.Logger;
+import com.google.common.base.Splitter;
+import com.google.common.collect.Lists;
+
/**
* A SortedKeyValueIterator that combines the Values for different versions (timestamp) of a Key within a row into a single Value. Combiner will replace one or
* more versions of a Key and their Values with the most recent Key and a Value which is the result of the reduce method. An {@link Column}
@@ -235,7 +235,7 @@ public abstract class Combiner extends WrappingIterator implements OptionDescrib
if (encodedColumns.length() == 0)
throw new IllegalArgumentException("The " + COLUMNS_OPTION + " must not be empty");
- combiners = new ColumnSet(Splitter.on(",").splitToList(encodedColumns));
+ combiners = new ColumnSet(Lists.newArrayList(Splitter.on(",").split(encodedColumns)));
}
@Override
@@ -279,7 +279,7 @@ public abstract class Combiner extends WrappingIterator implements OptionDescrib
if (encodedColumns.length() == 0)
throw new IllegalArgumentException("empty columns specified in option " + COLUMNS_OPTION);
- for (String columns : Splitter.on(",").splitToList(encodedColumns)) {
+ for (String columns : Splitter.on(",").split(encodedColumns)) {
if (!ColumnSet.isValidEncoding(columns))
throw new IllegalArgumentException("invalid column encoding " + encodedColumns);
}