[GitHub] [hudi] satishkotha commented on pull request #2129: [HUDI-1302] Add support for timestamp field in HiveSync
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
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
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
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