Re: [I] [FEATURE] Support table format output for Audit command [gravitino]

[via GitHub]

Abyss-lord commented on issue #5746:
URL: https://github.com/apache/gravitino/issues/5746#issuecomment-2658397703

   > > > Hi [@Abyss-lord](https://github.com/Abyss-lord), sure you can work on 
it.
   > > 
   > > 
   > > May I have your email, I have some to discuss.
   > 
   > Hi [@Abyss-lord](https://github.com/Abyss-lord), We can discuss here. Or I 
see you are in opensource4you Slack channel, If you have something to discuss, 
my ID is Chia-Ying Lee. You can dm me.
   
   I have a idea that we should standardize the output handling by 
consolidating all output methods at the Command layer. like this
   
   https://files.slack.com/files-pri/T05DWGA9CBT-F08D9SNQVPU/image.png
   
   ```java
   public void printInformation(String message) {
       if (context.quiet()) {
         return;
       }
   
       printResult(message);
     }
     
   
     protected <T> void printResults(T entity) {
       if (outputFormat == null) {
         PlainFormat.output(entity);
         return;
       }
   
       if (outputFormat.equals(OUTPUT_FORMAT_TABLE)) {
         TableFormat.output(entity);
       } else if (outputFormat.equals(OUTPUT_FORMAT_PLAIN)) {
         PlainFormat.output(entity);
       } else {
         throw new IllegalArgumentException("Unsupported output format");
       }
     }
   
     protected void printResults(String message) {
       PlainFormat.output(message);
     }
   ```
   This way, each subclass only needs to change the `getOutput` method, which 
makes unit testing easier, like this:
   
   
![Image](https://github.com/user-attachments/assets/fe82c702-0e4a-43f0-894b-ddf6f69bee29)
   
   


-- 
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: commits-unsubscr...@gravitino.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org