[
https://issues.apache.org/jira/browse/ACCUMULO-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15798999#comment-15798999
]
Luis Tavarez commented on ACCUMULO-4057:
----------------------------------------
You are welcome!
> Duplicated code in IteratorUtil
> -------------------------------
>
> Key: ACCUMULO-4057
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4057
> Project: Accumulo
> Issue Type: Improvement
> Reporter: Josh Elser
> Assignee: Luis Tavarez
> Priority: Trivial
> Fix For: 2.0.0
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> Duplicated code in
> https://github.com/apache/accumulo/blob/master/core/src/main/java/org/apache/accumulo/core/iterators/IteratorUtil.java#L236
> {code}
> public static <K extends WritableComparable<?>,V extends Writable>
> SortedKeyValueIterator<K,V> loadIterators(IteratorScope scope,
> SortedKeyValueIterator<K,V> source, KeyExtent extent,
> AccumuloConfiguration conf, List<IterInfo> ssiList,
> Map<String,Map<String,String>> ssio,
> IteratorEnvironment env, boolean useAccumuloClassLoader) throws
> IOException {
> List<IterInfo> iters = new ArrayList<IterInfo>(ssiList);
> Map<String,Map<String,String>> allOptions = new
> HashMap<String,Map<String,String>>();
> parseIteratorConfiguration(scope, iters, ssio, allOptions, conf);
> return loadIterators(source, iters, allOptions, env,
> useAccumuloClassLoader, conf.get(Property.TABLE_CLASSPATH));
> }
> public static <K extends WritableComparable<?>,V extends Writable>
> SortedKeyValueIterator<K,V> loadIterators(IteratorScope scope,
> SortedKeyValueIterator<K,V> source, KeyExtent extent,
> AccumuloConfiguration conf, List<IterInfo> ssiList,
> Map<String,Map<String,String>> ssio,
> IteratorEnvironment env, boolean useAccumuloClassLoader, String
> classLoaderContext) throws IOException {
> List<IterInfo> iters = new ArrayList<IterInfo>(ssiList);
> Map<String,Map<String,String>> allOptions = new
> HashMap<String,Map<String,String>>();
> parseIteratorConfiguration(scope, iters, ssio, allOptions, conf);
> return loadIterators(source, iters, allOptions, env,
> useAccumuloClassLoader, classLoaderContext);
> }
> {code}
> I thought I had commented on https://github.com/apache/accumulo/pull/51 about
> this, but maybe I forgot.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
