Jackie-Jiang opened a new issue #6240:
URL: https://github.com/apache/incubator-pinot/issues/6240


   Epoch time format has the following components: `[size]:[unit]:EPOCH`
   E.g. `1:DAYS:EPOCH`, `5:MINUTES:EPOCH`
   
   This size info is not preserved In the `SegmentGeneratorConfig` and 
`SegmentMetadata`, thus the generated segment won't have the correct time 
metadata when the size is not 1.
   
   We have never seen a use case with size other than 1, so we might just want 
to drop the size info from the time format. Also, the unit info does not apply 
to the simple-date-format, so I would recommend changing the time format to:
   `EPOCH:[unit]` for epoch time and `SIMPLE_DATE_FORMAT:[format]` for 
simple-date-format.


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

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