abhioncbr opened a new pull request, #10990: URL: https://github.com/apache/pinot/pull/10990
Previously, we skipped to set the value for min/max if the value was invalid. We were determining invalid values based on the following checks - If value contains leading or trailing whitespace character. - if the value contains `,` - if the value length is greater than 512. Changes in this PR are intended to ensure we will set the value of the segment's min/max property. It's part of the fix for the [issue](10793) and as per the discussion in the other [PR](https://github.com/apache/pinot/pull/10891) cc: @Jackie-Jiang -- 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
