mdmarshmallow commented on pull request #509:
URL: https://github.com/apache/lucene/pull/509#issuecomment-990443067


   The current way of storing ranges is a `Map<String, OrdRange>` that maps 
dimensions to their respective ord ranges. That obviously won't work anymore 
since want the ranges of any arbitrary path, not just a single dimension. I 
stored the new ord ranges in a tree which is what `HierarchicalOrdRange` really 
is, just a tree that stores ordinal ranges for paths. Maybe SSDVReaderState 
wasn't the best place to put it?


-- 
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

Reply via email to