(gravitino) branch main updated: Used passed in lineSeparator in writeHorizontalLine (#7998)

[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 75a05ff43e Used passed in lineSeparator in writeHorizontalLine (#7998)
75a05ff43e is described below

commit 75a05ff43e8b2f4e7fb29448712fcbaab277adb1
Author: Aman Chandra Kumar 
<115037337+spiritedswords...@users.noreply.github.com>
AuthorDate: Tue Aug 12 10:31:21 2025 +0530

    Used passed in lineSeparator in writeHorizontalLine (#7998)
    
    In
    clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java
    
    This update ensures that the lineSeparator argument passed to
    writeHorizontalLine is actually used instead of being ignored.
    Previously, the method always used System.lineSeparator(). Now it
    directly writes the provided `lineSeparator`.
    
    Closes #7978
---
 .../src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git 
a/clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java 
b/clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java
index b9c155ca87..bb5f5366ab 100644
--- 
a/clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java
+++ 
b/clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java
@@ -351,9 +351,7 @@ public abstract class TableFormat<T> extends 
BaseOutputFormat<T> {
       osw.write(right);
     }
 
-    if (lineSeparator != null) {
-      osw.write(System.lineSeparator());
-    }
+    osw.write(lineSeparator != null ? lineSeparator : System.lineSeparator());
   }
 
   /**
@@ -434,7 +432,7 @@ public abstract class TableFormat<T> extends 
BaseOutputFormat<T> {
       osw.write(right);
     }
 
-    osw.write(lineSeparator);
+    osw.write(lineSeparator != null ? lineSeparator : System.lineSeparator());
   }
 
   /**