Fix coverage issue for #3063 (#3155) (#3156)

(cherry picked from commit 95a1643) Signed-off-by: Tomoyuki Morita <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Nov 14, 2024 · b1caefb · b1caefb
1 parent 36b46bb
commit b1caefb
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/protocol/src/main/java/org/opensearch/sql/protocol/response/format/FlatResponseBase.java b/protocol/src/main/java/org/opensearch/sql/protocol/response/format/FlatResponseBase.java
@@ -84,7 +84,7 @@ protected List<List<String>> formatData(List<List<String>> lines) {
 
   protected String quoteIfRequired(String separator, String cell) {
     final String quote = "\"";
-    if (cell != null && (cell.contains(separator) || cell.contains(quote))) {
+    if (cell.contains(separator) || cell.contains(quote)) {
       return quote + cell.replaceAll(quote, quote + quote) + quote;
     } else {
       return cell;