[SQL] Do not serialize as JSON the contents of constant collections

mihaibudiu · mihaibudiu · commit e7336868c329 · 2026-04-11T16:44:12.000-07:00
Signed-off-by: Mihai Budiu &lt;mbudiu@feldera.com&gt;
diff --git a/sql-to-dbsp-compiler/SQL-compiler/src/main/java/org/dbsp/util/RelJsonWriter.java b/sql-to-dbsp-compiler/SQL-compiler/src/main/java/org/dbsp/util/RelJsonWriter.java
@@ -60,7 +60,7 @@ public RelJsonWriter(JsonBuilder jsonBuilder,
         this.verbosity = verbosity;
     }
 
-    final Set<String> highVerbosity = Set.of("exprs", "aggs", "condition");
+    final Set<String> highVerbosity = Set.of("exprs", "aggs", "condition", "tuples");
 
     protected void explain_(RelNode rel, List<Pair<String, Object>> values) {
         Map<String, Object> map = this.jsonBuilder.map();
diff --git a/sql-to-dbsp-compiler/SQL-compiler/src/test/java/org/dbsp/sqlCompiler/compiler/sql/MetadataTests.java b/sql-to-dbsp-compiler/SQL-compiler/src/test/java/org/dbsp/sqlCompiler/compiler/sql/MetadataTests.java
@@ -2134,4 +2134,17 @@ public void testSilenceWarning() {
             CREATE VIEW V AS SELECT x FROM T;""");
         Assert.assertEquals(0, cc.compiler.messages.messages.size());
     }
+
+    @Test
+    public void issue5896() throws IOException, SQLException {
+        String sql = "CREATE VIEW v AS select cot(0)";
+        File file = createInputScript(sql);
+        File json = this.createTempJsonFile();
+        CompilerMessages msg = CompilerMain.execute(
+                "--dataflow", json.getPath(), "--noRust", file.getPath());
+        Assert.assertEquals(0, msg.exitCode);
+        ObjectMapper mapper = Utilities.deterministicObjectMapper();
+        JsonNode parsed = mapper.readTree(json);
+        Assert.assertNotNull(parsed);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public RelJsonWriter(JsonBuilder jsonBuilder,`
`60`	`60`	`this.verbosity = verbosity;`
`61`	`61`	`}`
`62`	`62`
`63`		`- final Set<String> highVerbosity = Set.of("exprs", "aggs", "condition");`
	`63`	`+ final Set<String> highVerbosity = Set.of("exprs", "aggs", "condition", "tuples");`
`64`	`64`
`65`	`65`	`protected void explain_(RelNode rel, List<Pair<String, Object>> values) {`
`66`	`66`	`Map<String, Object> map = this.jsonBuilder.map();`