saurabhd336 commented on PR #12568: URL: https://github.com/apache/pinot/pull/12568#issuecomment-1980689303
cc: @itschrispeck and @chenboat for reviewing changes related to mutable json index moving to TreeMap since it changes the implementation of json_extract_index functions. 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. The implementation of range and regexp evaluation also benefits from this move. -- 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
