[
https://issues.apache.org/jira/browse/ACCUMULO-3948?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000818#comment-15000818
]
ASF GitHub Bot commented on ACCUMULO-3948:
------------------------------------------
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/51#discussion_r44564725
--- Diff:
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/ScanDataSource.java
---
@@ -178,8 +178,13 @@ public boolean isCurrent() {
VisibilityFilter visFilter = new VisibilityFilter(colFilter,
options.getAuthorizations(), options.getDefaultLabels());
- return
iterEnv.getTopLevelIterator(IteratorUtil.loadIterators(IteratorScope.scan,
visFilter, tablet.getExtent(), tablet.getTableConfiguration(),
- options.getSsiList(), options.getSsio(), iterEnv));
+ if (null == options.getContext()) {
--- End diff --
Why not push down null handling to `IteratorUtil.loadIterators`?
> Enable A/B testing of scan iterators on a table
> -----------------------------------------------
>
> Key: ACCUMULO-3948
> URL: https://issues.apache.org/jira/browse/ACCUMULO-3948
> Project: Accumulo
> Issue Type: Improvement
> Components: tserver
> Reporter: Dave Marion
> Assignee: Dave Marion
> Fix For: 1.8.0
>
> Attachments: ACCUMULO-3948.1-6-3.patch, TestIterator.patch
>
>
> Classpath contexts are assigned to a table via the table configuration. You
> can test at scale by cloning your table and assigning a new classpath context
> to the cloned table. However, you would also need to change your application
> to use the new table names and since we cannot disable compactions you would
> start to consume more space in the filesystem for that table. We can support
> users passing in a context name to use for the scan on existing tables.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
