zhangyue19921010 edited a comment on pull request #4453: URL: https://github.com/apache/hudi/pull/4453#issuecomment-1003842005
The root cause of `hms and jdbc fails.` is https://github.com/apache/hudi/pull/4452 changed the logic that `isDropPartition`do not need to check schema chenged. ``` if (!isDropPartition) { xxx syncSchema(); xxx } ``` Here are the related stack trace ``` Caused by: org.apache.hudi.hive.HoodieHiveSyncException: Failed to update table for test1 at org.apache.hudi.hive.ddl.HMSDDLExecutor.updateTableDefinition(HMSDDLExecutor.java:150) at org.apache.hudi.hive.HoodieHiveClient.updateTableDefinition(HoodieHiveClient.java:205) at org.apache.hudi.hive.HiveSyncTool.syncSchema(HiveSyncTool.java:252) at org.apache.hudi.hive.HiveSyncTool.syncHoodieTable(HiveSyncTool.java:184) at org.apache.hudi.hive.HiveSyncTool.doSync(HiveSyncTool.java:129) at org.apache.hudi.hive.HiveSyncTool.syncHoodieTable(HiveSyncTool.java:115) ... 123 more ``` Just raise a pr to fix the test with adding the schema info in fake replace commit file. https://github.com/apache/hudi/pull/4493 It's passed on my local env. @nsivabalan Would you mind to take look? Thanks a lot. -- 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]
