You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2020/01/11 05:45:54 UTC
[GitHub] [calcite] XuQianJin-Stars commented on a change in pull request
#1143: [CALCITE-2884] Implement JSON_INSERT, JSON_REPLACE, JSON_SET
XuQianJin-Stars commented on a change in pull request #1143: [CALCITE-2884] Implement JSON_INSERT, JSON_REPLACE, JSON_SET
URL: https://github.com/apache/calcite/pull/1143#discussion_r365502033
##########
File path: core/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java
##########
@@ -1340,6 +1340,15 @@ public boolean argumentMustBeScalar(int ordinal) {
@Deprecated // to be removed before 2.0
public static final SqlFunction JSON_STORAGE_SIZE = SqlLibraryOperators.JSON_STORAGE_SIZE;
+ @Deprecated // to be removed before 2.0
+ public static final SqlFunction JSON_INSERT = SqlLibraryOperators.JSON_INSERT;
+
+ @Deprecated // to be removed before 2.0
+ public static final SqlFunction JSON_REPLACE = SqlLibraryOperators.JSON_REPLACE;
Review comment:
> Revert the change.
Currently, it cannot be temporarily deleted. I will modify the entire JSON function later.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services