DanielZhu58 commented on code in PR #5578: URL: https://github.com/apache/hive/pull/5578#discussion_r1901410492
########## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java: ########## @@ -7420,6 +7420,64 @@ public boolean delete_partition_column_statistics(String dbName, String tableNam return ret; } + @Override + public boolean delete_partition_column_statistics_req(DeletePartitionColumnStatisticsRequest req) throws TException { + String dbName = req.getDb_name(); + String tableName = req.getTbl_name(); + List<String> colNames = req.getCol_names(); + String partName = req.getPart_name(); + String engine = req.getEngine(); + dbName = dbName.toLowerCase(); + String[] parsedDbName = parseDbName(dbName, conf); + tableName = tableName.toLowerCase(); + if (colNames != null) { + for (String colName : colNames) { + colName = colName.toLowerCase(); + startFunction("delete_column_statistics_by_partition",": table=" + + TableName.getQualified(parsedDbName[CAT_NAME], parsedDbName[DB_NAME], tableName) + + " partition=" + partName + " column=" + colName); + } + } + boolean ret = false, committed = false; + + getMS().openTransaction(); + try { + List<String> partVals = getPartValsFromName(getMS(), parsedDbName[CAT_NAME], parsedDbName[DB_NAME], tableName, partName); + Table table = getMS().getTable(parsedDbName[CAT_NAME], parsedDbName[DB_NAME], tableName); + // This API looks unused; if it were used we'd need to update stats state and write ID. + // We cannot just randomly nuke some txn stats. + if (TxnUtils.isTransactionalTable(table)) { Review Comment: This comments are not written by me. We can mark this issue as to do, and discuss about it later. -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org