[GitHub] [hudi] satishkotha commented on pull request #2129: [HUDI-1302] Add support for timestamp field in HiveSync

2020-10-09 Thread GitBox 


satishkotha commented on pull request #2129:
URL: https://github.com/apache/hudi/pull/2129#issuecomment-706341395


   @n3nash @pratyakshsharma @bschell can any of you review this?



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

[GitHub] [hudi] satishkotha commented on pull request #2129: [HUDI-1302] Add support for timestamp field in HiveSync

2020-10-07 Thread GitBox 


satishkotha commented on pull request #2129:
URL: https://github.com/apache/hudi/pull/2129#issuecomment-704748964


   @pratyakshsharma @n3nash Please take a look.



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

[GitHub] [hudi] satishkotha commented on pull request #2129: [HUDI-1302] Add support for timestamp field in HiveSync

2020-10-01 Thread GitBox 


satishkotha commented on pull request #2129:
URL: https://github.com/apache/hudi/pull/2129#issuecomment-702275562


   @pratyakshsharma will you be able to review this week?



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

[GitHub] [hudi] satishkotha commented on pull request #2129: [HUDI-1302] Add support for timestamp field in HiveSync

2020-09-29 Thread GitBox 


satishkotha commented on pull request #2129:
URL: https://github.com/apache/hudi/pull/2129#issuecomment-700819993


   > Can you test on Hive to check if Hive can read data with hudi avro 
timestamp type?
   > I think there are more test cases and other things to let Hive support 
timestamp type.
   
   Yes, additional changes are needed in Hive. For short term, we made a 
temporary patch to Hive to make it work (@s-sanjay plans to open a draft PR 
soon). 
   I believe Hive4 also supports this in a different way.



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