[AMORO-4099] Add table-summary metric collection option for non-optimizing tables

- Allow collecting table_summary metrics when self-optimizing is disabled
  by setting table-summary.enabled=true
- Fix periodic collection bug: remove optimizingNotNecessary() call in
  summary-only branch to prevent snapshot gate from blocking subsequent
  collections
- Separate property key from self-optimizing prefix:
  self-optimizing.table-summary.enabled -> table-summary.enabled
- Add debug logging for table summary collection path
- Add comprehensive test coverage for summary-only mode

Signed-off-by: Jiwon Park <jpark92@outlook.kr>

Author: j1wonpark

amoro-ams/src/main/java/org/apache/amoro/server/scheduler/inline/TableRuntimeRefreshExecutor.java

@@ -61,11 +61,10 @@ protected long getNextExecutingTime(TableRuntime tableRuntime) {
   }
 
   private void tryEvaluatingPendingInput(DefaultTableRuntime tableRuntime, MixedTable table) {
-    // only evaluate pending input when optimizing is enabled and in idle state
     OptimizingConfig optimizingConfig = tableRuntime.getOptimizingConfig();
     if (optimizingConfig.isEnabled()
         && tableRuntime.getOptimizingStatus().equals(OptimizingStatus.IDLE)) {
-
+      // Evaluate pending input and collect table summary when optimizing is enabled and idle
       if (optimizingConfig.isMetadataBasedTriggerEnabled()
           && !MetadataBasedEvaluationEvent.isEvaluatingNecessary(
               optimizingConfig, table, tableRuntime.getLastPlanTime())) {
@@ -89,6 +88,16 @@ private void tryEvaluatingPendingInput(DefaultTableRuntime tableRuntime, MixedTa
         tableRuntime.optimizingNotNecessary();
       }
       tableRuntime.setTableSummary(evaluator.getPendingInput());
+    } else if (!optimizingConfig.isEnabled() && optimizingConfig.isTableSummaryEnabled()) {
+      // Collect table summary metrics even when optimizing is disabled
+      logger.info(
+          "{} collecting table summary (optimizing disabled, tableSummary enabled)",
+          tableRuntime.getTableIdentifier());
+      AbstractOptimizingEvaluator evaluator =
+          IcebergTableUtil.createOptimizingEvaluator(tableRuntime, table, maxPendingPartitions);
+      AbstractOptimizingEvaluator.PendingInput summary = evaluator.getPendingInput();
+      logger.info("{} table summary collected: {}", tableRuntime.getTableIdentifier(), summary);
+      tableRuntime.setTableSummary(summary);
     }
   }
 
@@ -122,12 +131,26 @@ public void execute(TableRuntime tableRuntime) {
       AmoroTable<?> table = loadTable(tableRuntime);
       defaultTableRuntime.refresh(table);
       MixedTable mixedTable = (MixedTable) table.originalTable();
-      if ((mixedTable.isKeyedTable()
-              && (lastOptimizedSnapshotId != defaultTableRuntime.getCurrentSnapshotId()
-                  || lastOptimizedChangeSnapshotId
-                      != defaultTableRuntime.getCurrentChangeSnapshotId()))
-          || (mixedTable.isUnkeyedTable()
-              && lastOptimizedSnapshotId != defaultTableRuntime.getCurrentSnapshotId())) {
+      boolean snapshotChanged =
+          (mixedTable.isKeyedTable()
+                  && (lastOptimizedSnapshotId != defaultTableRuntime.getCurrentSnapshotId()
+                      || lastOptimizedChangeSnapshotId
+                          != defaultTableRuntime.getCurrentChangeSnapshotId()))
+              || (mixedTable.isUnkeyedTable()
+                  && lastOptimizedSnapshotId != defaultTableRuntime.getCurrentSnapshotId());
+      OptimizingConfig optimizingConfig = defaultTableRuntime.getOptimizingConfig();
+      boolean tableSummaryOnly =
+          !optimizingConfig.isEnabled() && optimizingConfig.isTableSummaryEnabled();
+      logger.info(
+          "{} refresh check: snapshotChanged={}, optimizingEnabled={}, tableSummaryEnabled={}, "
+              + "lastOptimizedSnapshotId={}, currentSnapshotId={}",
+          tableRuntime.getTableIdentifier(),
+          snapshotChanged,
+          optimizingConfig.isEnabled(),
+          optimizingConfig.isTableSummaryEnabled(),
+          lastOptimizedSnapshotId,
+          defaultTableRuntime.getCurrentSnapshotId());
+      if (snapshotChanged || tableSummaryOnly) {
         tryEvaluatingPendingInput(defaultTableRuntime, mixedTable);
       }
     } catch (Throwable throwable) {

amoro-ams/src/main/java/org/apache/amoro/server/table/TableConfigurations.java

@@ -232,6 +232,11 @@ public static OptimizingConfig parseOptimizingConfig(Map<String, String> propert
                 properties,
                 TableProperties.ENABLE_SELF_OPTIMIZING,
                 TableProperties.ENABLE_SELF_OPTIMIZING_DEFAULT))
+        .setTableSummaryEnabled(
+            CompatiblePropertyUtil.propertyAsBoolean(
+                properties,
+                TableProperties.TABLE_SUMMARY_ENABLED,
+                TableProperties.TABLE_SUMMARY_ENABLED_DEFAULT))
         .setAllowPartialCommit(
             CompatiblePropertyUtil.propertyAsBoolean(
                 properties,