gsmiller commented on pull request #509: URL: https://github.com/apache/lucene/pull/509#issuecomment-990419374
Thanks for taking a pass at this! I've started looking at the change but hope to have more time tomorrow to dive into details and start leaving feedback. One thing that's not clear to me up front is why ordinal range state needs to be hierarchical (i.e., `HierarchicalOrdRange`) but maybe that will become more clear as I'm able to spend more time on the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
