gsmiller commented on pull request #509: URL: https://github.com/apache/lucene/pull/509#issuecomment-990550171
@mdmarshmallow right got it. I wonder if an actual tree is really necessary of if it's really just still a map of "path" -> ordRange? There's no need to traverse the "tree" structure of this right? Just do a simple key/value lookup of a fully-specified path to it's ordinal ranges? (e.g., there's no need to recursively traverse to get all ordinal ranges for everything under a given root, etc. right?) Let me get into the review a bit more tomorrow though and get a better idea of what's going on before I comment further :) -- 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
