hongkunxu opened a new pull request, #17467: URL: https://github.com/apache/pinot/pull/17467
In SegmentProcessorUtils, fields are classified and sorted when collecting all FieldSpecs. However, there is a logical issue in the current implementation: although fields are added in the intended order, the sorting of nonMetricFieldSpecs happens after they are added to the final list, so the sort has no effect. While this does not currently impact downstream behavior, the logic is incorrect and potentially misleading. This PR fixes the ordering to ensure the sorting is applied as intended. Hi @Jackie-Jiang @xiangfu0 , please help review this PR when you are free. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
