codope commented on pull request #3203:
URL: https://github.com/apache/hudi/pull/3203#issuecomment-963247562


   > @nsivabalan @codope for this test case 
https://gist.github.com/nsivabalan/950bdde0b3b64349bf0395b2e3e928d8, when sync 
to hive , Why use "_ hoodie_ partition_ Path" as partition instead of "driver". 
L102 in HoodieParquetRealtimeInputFormat will add "_ hoodie_ partition_ Path" 
in RequiedProjectionFields the method 
HoodieRealtimeRecordReaderUtils.orderFields will remove "_ hoodie_ partition_ 
Path" from projectCols since partitionFiedls contains this col. then L264 will 
throw Exception. do you think we need fixed that problem, If necessary, I will 
raise an additional JIRA to track this issue
   
   Good point @xiarixiaoyao 
   Will #3630 fix 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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to