leesf commented on a change in pull request #4291:
URL: https://github.com/apache/hudi/pull/4291#discussion_r767475656
##########
File path:
hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncTool.java
##########
@@ -166,20 +165,28 @@ protected void syncHoodieTable(String tableName, boolean
useRealtimeInputFormat,
// Check if the necessary table exists
boolean tableExists = hoodieHiveClient.doesTableExist(tableName);
- // Get the parquet schema for this table looking at the latest commit
- MessageType schema = hoodieHiveClient.getDataSchema();
-
- // Currently HoodieBootstrapRelation does support reading bootstrap MOR rt
table,
- // so we disable the syncAsSparkDataSourceTable here to avoid read such
kind table
- // by the data source way (which will use the HoodieBootstrapRelation).
- // TODO after we support bootstrap MOR rt table in
HoodieBootstrapRelation[HUDI-2071], we can remove this logical.
- if (hoodieHiveClient.isBootstrap()
- && hoodieHiveClient.getTableType() == HoodieTableType.MERGE_ON_READ
- && !readAsOptimized) {
- cfg.syncAsSparkDataSourceTable = false;
+ // check if isDeletePartition
+ boolean isDeletePartition = hoodieHiveClient.isDeletePartition();
Review comment:
rename to `isDropPartition` and change to isDropPartition method name
--
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: commits-unsubscr...@hudi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
