mghildiy commented on issue #13004: URL: https://github.com/apache/pinot/issues/13004#issuecomment-2481077427
Going through the code for DateTimeFieldSpec and associated classes like DateTimeFormatPatternSpec etc, there is TimeFormat which allows users to set format in which user inputs value for this data type. It currently has three options: EPOCH, TIMESTAMP, SIMPLE_DATE_FORMAT. For first 2, we set default timezone of UTC. SIMPLE_DATE_FORMAT allows users to provde value for timezone too. I assume same behaviour for TimestampWithTimeZone is to be exhibited. Also, I would add a new implementation in PinotDataType for TimestampWithTimeZone which would convert from source value to internal value with timezone id packed. -- 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
