[GitHub] Jackie-Jiang opened a new pull request #3895: Fix the RecordReader to pick incoming time column

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

Jackie-Jiang opened a new pull request #3895: Fix the RecordReader to pick incoming time column
URL: https://github.com/apache/incubator-pinot/pull/3895
 
 
   Fix AvroRecordReader, CSVRecordReader, JSONRecordReader to be able to read incoming time column
   - If incoming and outgoing time column name are the same, use incoming field spec (data type)
   - If incoming and outgoing time column name are different, try reading both of them
   - If no value provided, do not fill default value for time column but throw exception (don't allow default time column)
   
   Also fix the issue where AvroRecordReader does not convert the value into data type from schema
   Add tests for all scenarios

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org