itschrispeck commented on PR #12568: URL: https://github.com/apache/pinot/pull/12568#issuecomment-1982602428
> IMO, iterating over all the keys of the mutable json index's map is wasteful, and we can do better by filtering the keys to iterate over by using a TreeMap. +1 we thought about this but weren't sure how much insert/exact match performance would be impacted. I am happy to see this change, TreeMap fits our typical use cases much better! -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
