[GitHub] [pinot] walterddr edited a comment on issue #7786: StringColumnPreIndexStatsCollector doesnt work for datetime

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

walterddr edited a comment on issue #7786:
URL: https://github.com/apache/pinot/issues/7786#issuecomment-972995918


   that's definitely one solution but IMO it will make user experience bad. either we enhance our documentation and error messaging; or we should automatically populate a shadow timeColumn that's sorted by time. Especially when we have the tableConfig / ingestionConfig telling us we should sort by underlying datetime.
   
   also could you elaborate more on this
   
   > We process date time fields the same way as dimensions and metrics, so the ordering must be aligned with the data type. Ordering date time field differently will cause wrong result.
   
   which part specifically will fail?


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