Skip to content

OAK-1977 ContentMirrorStoreStrategy to utilize path restriction if avail...#23

Closed
catholicon wants to merge 2 commits intoapache:1.0from
catholicon:path-aware-contentMirrorIndex
Closed

OAK-1977 ContentMirrorStoreStrategy to utilize path restriction if avail...#23
catholicon wants to merge 2 commits intoapache:1.0from
catholicon:path-aware-contentMirrorIndex

Conversation

@catholicon
Copy link
Contributor

proposed patch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section would come into play only when entryCount isn't available on the indexed value node. This is for cost optimzation. Actual query execution would still be improved.

@catholicon
Copy link
Contributor Author

I'm still figuring out some API and how to use them... closing this pull request for now... sorry :(

@catholicon catholicon closed this Jul 21, 2014
@catholicon catholicon reopened this Jul 21, 2014
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section would come into play only when entryCount isn't available on the indexed value node. This is for cost optimzation. Actual query execution would still be improved.

@catholicon
Copy link
Contributor Author

I don't why Travis CI is stuck. The branch passed mvn clean install -PintegrationTesting though on my system.

@catholicon
Copy link
Contributor Author

Closing pull request... would submit patch against trunk in the issue itself.

@catholicon catholicon closed this Jul 22, 2014
@catholicon catholicon deleted the path-aware-contentMirrorIndex branch July 22, 2014 06:26
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.

1 participant