shaofengshi commented on PR #6198:
URL: https://github.com/apache/gravitino/pull/6198#issuecomment-2589307862

   > What is the usage of this variable, can we elaborate the usage scenario 
for this? Also the change seems too fragmented, we need to change the code 
everywhere, A better way is to define an `output` class and can be controlled 
in there, rather than adding system output everywhere.
   > 
   > Besides, if we don't want output, we can simply use ` /foo/bar 2&1 > 
/dev/null`, which is a typical usage pattern in *nix, I'm curious what is the 
benefit of adding this.
   
   +1; the current implementation is not good.


-- 
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