review-2

Author: dengzhhu653

standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java

@@ -4614,8 +4614,8 @@ public boolean delete_column_statistics_req(DeleteColumnStatisticsRequest req) t
         ret = rawStore.deleteTableColumnStatistics(parsedDbName[CAT_NAME], parsedDbName[DB_NAME], tableName, colNames, engine);
         if (ret) {
           eventType = EventType.DELETE_TABLE_COLUMN_STAT;
-          for (String colName :
-              colNames == null ? table.getSd().getCols().stream().map(FieldSchema::getName).toList() : colNames) {
+          for (String colName : colNames == null || colNames.isEmpty() ?
+              table.getSd().getCols().stream().map(FieldSchema::getName).toList() : colNames) {
             if (transactionalListeners != null && !transactionalListeners.isEmpty()) {
               MetaStoreListenerNotifier.notifyEvent(transactionalListeners, eventType,
                   new DeleteTableColumnStatEvent(parsedDbName[CAT_NAME], parsedDbName[DB_NAME], tableName, colName, engine, this));
@@ -4638,8 +4638,8 @@ public boolean delete_column_statistics_req(DeleteColumnStatisticsRequest req) t
                 partNames, colNames, engine);
         if (ret) {
           eventType = EventType.DELETE_PARTITION_COLUMN_STAT;
-          for (String colName : colNames == null ? table.getSd().getCols().stream().map(FieldSchema::getName)
-              .toList() : colNames) {
+          for (String colName : colNames == null || colNames.isEmpty() ?
+              table.getSd().getCols().stream().map(FieldSchema::getName).toList() : colNames) {
             for (String partName : partNames) {
               List<String> partVals = getPartValsFromName(table, partName);
               if (transactionalListeners != null && !transactionalListeners.isEmpty()) {