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

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


The following commit(s) were added to refs/heads/main by this push:
     new fd6aaa59ba [CELEBORN-2396][CLI] Fix CelebornCli.main discarding the 
configured CommandLine instance
fd6aaa59ba is described below

commit fd6aaa59ba4ed796db9e4373fc31fde5dfb0b3f3
Author: yew1eb <[email protected]>
AuthorDate: Mon Aug 3 10:37:52 2026 +0800

    [CELEBORN-2396][CLI] Fix CelebornCli.main discarding the configured 
CommandLine instance
    
    ### What changes were proposed in this pull request?
    
    Call `execute()` on the configured `CommandLine` instance instead of 
creating and executing a new one.
    
    ### Why are the changes needed?
    
    `CelebornCli.main` creates a `CommandLine`, sets case-sensitivity options 
on it, but then executes a brand-new instance, so the configuration is silently 
discarded and the two setter calls are dead code.
    
    ### Does this PR introduce any user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Closes #3773 from yew1eb/CELEBORN-2396.
    
    Authored-by: yew1eb <[email protected]>
    Signed-off-by: Nicholas Jiang <[email protected]>
---
 cli/src/main/scala/org/apache/celeborn/cli/CelebornCli.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cli/src/main/scala/org/apache/celeborn/cli/CelebornCli.scala 
b/cli/src/main/scala/org/apache/celeborn/cli/CelebornCli.scala
index e1b9444ae0..f73436095f 100644
--- a/cli/src/main/scala/org/apache/celeborn/cli/CelebornCli.scala
+++ b/cli/src/main/scala/org/apache/celeborn/cli/CelebornCli.scala
@@ -40,6 +40,6 @@ object CelebornCli {
     val cmd = new CommandLine(new CelebornCli())
     cmd.setOptionsCaseInsensitive(false)
     cmd.setSubcommandsCaseInsensitive(false)
-    new CommandLine(new CelebornCli()).execute(args: _*)
+    cmd.execute(args: _*)
   }
 }

Reply via email to