Skip to content

Support pre-filtering using Index materialized attributes.#244

Merged
groldan merged 3 commits into
locationtech:masterfrom
groldan:indexing_featurereader_pre-post-filtering
Feb 7, 2017
Merged

Support pre-filtering using Index materialized attributes.#244
groldan merged 3 commits into
locationtech:masterfrom
groldan:indexing_featurereader_pre-post-filtering

Conversation

@groldan

@groldan groldan commented Feb 6, 2017

Copy link
Copy Markdown
Member

When there're materialized attributes in an Index,
FeatureReaderBuilder creates an Iterator
that's pre-filtered using the available materialized
attribute values to the extent possible.

Signed-off-by: Gabriel Roldan groldan@boundlessgeo.com

@groldan groldan force-pushed the indexing_featurereader_pre-post-filtering branch 4 times, most recently from 2f39f0a to 566aacf Compare February 6, 2017 23:34
When there're materialized attributes in an Index,
FeatureReaderBuilder creates an Iterator<NodeRef>
that's pre-filtered using the available materialized
attribute values to the extent possible.

Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
@groldan groldan force-pushed the indexing_featurereader_pre-post-filtering branch from 566aacf to ec8bb3c Compare February 6, 2017 23:48
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
@groldan groldan force-pushed the indexing_featurereader_pre-post-filtering branch from 930569e to 4dd737d Compare February 7, 2017 01:09
Signed-off-by: dblasby <dblasby@boundlessgeo.com>
@groldan groldan force-pushed the indexing_featurereader_pre-post-filtering branch from 4dd737d to 28f804e Compare February 7, 2017 02:07
@groldan groldan merged commit 2af8fc4 into locationtech:master Feb 7, 2017
@groldan groldan deleted the indexing_featurereader_pre-post-filtering branch February 7, 2017 20:49
@groldan groldan modified the milestone: v1.1.0 Mar 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants