[GitHub] [pinot] Jackie-Jiang commented on pull request #9866: Do not sort the instances in InstancePartitions

[GitBox <gi...@apache.org>]

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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org