jainankitk opened a new pull request, #14476: URL: https://github.com/apache/lucene/pull/14476
### Description IMO, storing splitX for every `TreeNode` makes sense, only if we are exposing API to partially search `ComponentTree` from arbitrary `TreeNode` which is not allowed today. Hence, makes sense to address the TODO and remove `splitX`. Code without `splitX` looks reasonably straightforward to me, just follow the same structure as `createTree`. Start with `ROOT_SPLITX` and keep toggling `splitX` as we recurse through the levels <!-- If this is your first contribution to Lucene, please make sure you have reviewed the contribution guide. https://github.com/apache/lucene/blob/main/CONTRIBUTING.md --> -- 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
