You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by ch...@apache.org on 2021/11/17 06:52:59 UTC
[calcite] branch master updated: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder
This is an automated email from the ASF dual-hosted git repository.
chunwei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/master by this push:
new faee09f [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder
faee09f is described below
commit faee09f3d014659a024184800070f249b92af662
Author: chunwei.lcw <ch...@alibaba-inc.com>
AuthorDate: Mon Nov 15 14:32:26 2021 +0800
[CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder
---
.../java/org/apache/calcite/rel/externalize/RelJsonWriter.java | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/core/src/main/java/org/apache/calcite/rel/externalize/RelJsonWriter.java b/core/src/main/java/org/apache/calcite/rel/externalize/RelJsonWriter.java
index dc785f2..964a945 100644
--- a/core/src/main/java/org/apache/calcite/rel/externalize/RelJsonWriter.java
+++ b/core/src/main/java/org/apache/calcite/rel/externalize/RelJsonWriter.java
@@ -50,9 +50,13 @@ public class RelJsonWriter implements RelWriter {
//~ Constructors -------------------------------------------------------------
public RelJsonWriter() {
- jsonBuilder = new JsonBuilder();
- relList = jsonBuilder.list();
- relJson = new RelJson(jsonBuilder);
+ this(new JsonBuilder());
+ }
+
+ public RelJsonWriter(JsonBuilder jsonBuilder) {
+ this.jsonBuilder = jsonBuilder;
+ relList = this.jsonBuilder.list();
+ relJson = new RelJson(this.jsonBuilder);
}
//~ Methods ------------------------------------------------------------------