(gravitino) branch main updated: [#8730][Improvement] Fix CLI distribution output when distribution is none (#9066)

[jmclean]

This is an automated email from the ASF dual-hosted git repository.

jmclean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new ddcdd8ee83 [#8730][Improvement] Fix CLI distribution output when 
distribution is none (#9066)
ddcdd8ee83 is described below

commit ddcdd8ee836e212107e53a5406ff08211a4ede8d
Author: chanwonlee <116537544+chanwon...@users.noreply.github.com>
AuthorDate: Wed Nov 12 08:40:24 2025 +0900

    [#8730][Improvement] Fix CLI distribution output when distribution is none 
(#9066)
    
    ### What changes were proposed in this pull request?
    
    This PR fixes the CLI distribution output when the distribution strategy
    is none.
    
    ### Why are the changes needed?
    
    When distribution strategy is NONE, displaying the number is confusing
    and unnecessary. This improvement makes the output cleaner and more
    meaningful by only showing "NONE".
    
    Fix: #8730
    
    ### Does this PR introduce _any_ user-facing change?
    - Before: `NONE,0`
    - After: `NONE`
    
    ### How was this patch tested?
    N/A
---
 .../java/org/apache/gravitino/cli/commands/TableDistribution.java  | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/TableDistribution.java
 
b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/TableDistribution.java
index 5a24713e3d..cf091ecf69 100644
--- 
a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/TableDistribution.java
+++ 
b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/TableDistribution.java
@@ -22,6 +22,7 @@ package org.apache.gravitino.cli.commands;
 import org.apache.gravitino.NameIdentifier;
 import org.apache.gravitino.cli.CommandContext;
 import org.apache.gravitino.rel.expressions.distributions.Distribution;
+import org.apache.gravitino.rel.expressions.distributions.Strategy;
 
 /** Displays the details of a table's distirbution. */
 public class TableDistribution extends TableCommand {
@@ -59,6 +60,10 @@ public class TableDistribution extends TableCommand {
       exitWithError(exp.getMessage());
     }
 
-    printInformation(distribution.strategy() + "," + distribution.number());
+    if (distribution.strategy() == Strategy.NONE) {
+      printInformation(distribution.strategy().toString());
+    } else {
+      printInformation(distribution.strategy() + "," + distribution.number());
+    }
   }
 }