Jackie-Jiang commented on PR #9866: URL: https://github.com/apache/pinot/pull/9866#issuecomment-1329844531
> Do we have the context on why the sort was added at the first place? I'm good with removing the sort. It is there in the first version when the feature is added. I think it is sorted probably because it is easier for human read. The minimize movement feature doesn't return sorted instances, so I believe we don't rely on the list to be sorted. -- 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
