[ 
https://issues.apache.org/jira/browse/ACCUMULO-2639?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jeffrey Manno resolved ACCUMULO-2639.
-------------------------------------
    Resolution: Not A Problem

This area of the code has been cleaned up a bit since this was posted. Commit 
[https://github.com/apache/accumulo/commit/94bf129c8c3635bb99b272db009350a448503607#diff-54d15d7ecbb4d8d820e9ec2f4913224e3452b7b49993c847c848294a7a83c162]
 added a separate ScannerOptions class to be used to set column options.

> MetadataLocationObtainer opaquely sets ScannerOpts
> --------------------------------------------------
>
>                 Key: ACCUMULO-2639
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2639
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.6.0
>            Reporter: Mike Drob
>            Priority: Major
>
> {code}
>     ScannerOptions opts = new ScannerOptions() {
>       ScannerOptions setOpts() {
>         this.fetchedColumns = locCols;
>         this.serverSideIteratorList = new ArrayList<IterInfo>();
>         // see comment in lookupTablet about why iterator is used
>         this.serverSideIteratorList.add(new IterInfo(10000, 
> WholeRowIterator.class.getName(), "WRI"));
>         return this;
>       }
>     }.setOpts();
> {code}
> This code is not very clear and probably exists because we didn't have a good 
> API on ScannerOpts. This should be cleaned up.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to