Github user JamesRTaylor commented on the pull request:
https://github.com/apache/phoenix/pull/1#issuecomment-48825943
Related to the revert of the ScanRanges change, you'll need to make this
change to prevent the SkipRangeParallelIteratorRegionSplitter from being used
(as you always need to scan all regions for a local index). Cleanest might be
to just implement a simple ParallelIteratorRegionSplitter for use when a local
index is used that just returns all regions:
public class ParallelIteratorRegionSplitterFactory {
public static ParallelIteratorRegionSplitter
getSplitter(StatementContext context, TableRef table, HintNode hintNode) throws
SQLException {
if (!isLocalIndex &&
context.getScanRanges().useSkipScanFilter()) {
return
SkipRangeParallelIteratorRegionSplitter.getInstance(context, table, hintNode);
}
return
DefaultParallelIteratorRegionSplitter.getInstance(context, table, hintNode);
}
}
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
