raghavyadav01 opened a new pull request, #13897:
URL: https://github.com/apache/pinot/pull/13897

   Currently addColumnMinMaxValueInfo in SegmentColumnIndexCreator does not 
honor the column field maxLength for adding min/max values in metadata. If 
minString for a column is longer than default max limit 512 then minValue is 
set to null. This causes the segment to be preProcessed every time segment is 
reloaded. 
   
   Here is the summary of changes:
   - write - Use column maxLength property to add min/max in 
addColumnMinMaxValueInfo.
   - load -  check min / max separately and set them separately into the 
builder and populate accordingly in ColumneMetadataImpl.
   - Test -  Added a unit test to reproduce the issue and verify the fix. 
   
   


-- 
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

Reply via email to