yew1eb opened a new pull request, #3773:
URL: https://github.com/apache/celeborn/pull/3773

   ### 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?
   
   Manual review.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to