Add support for multiple Trino JSON functions

Adding support for clauses in JSON Functions. Fixes #2368

Before:
Result "net.sf.jsqlparser.benchmark.JSQLParserBenchmark.parseSQLStatements":
  33.555 ±(99.9%) 2.650 ms/op [Average]
  (min, avg, max) = (31.657, 33.555, 37.077), stdev = 2.479
  CI (99.9%): [30.905, 36.205] (assumes normal distribution)

After:
Result "net.sf.jsqlparser.benchmark.JSQLParserBenchmark.parseSQLStatements":
  36.242 ±(99.9%) 1.573 ms/op [Average]
  (min, avg, max) = (33.253, 36.242, 38.801), stdev = 2.100
  CI (99.9%): [34.669, 37.815] (assumes normal distribution)

Author: David Hayes

src/main/java/net/sf/jsqlparser/expression/ExpressionVisitor.java

@@ -658,6 +658,14 @@ default void visit(JsonFunction jsonFunction) {
         this.visit(jsonFunction, null);
     }
 
+    default <S> T visit(JsonTableFunction jsonTableFunction, S context) {
+        return visit((Function) jsonTableFunction, context);
+    }
+
+    default void visit(JsonTableFunction jsonTableFunction) {
+        this.visit(jsonTableFunction, null);
+    }
+
     <S> T visit(ConnectByRootOperator connectByRootOperator, S context);
 
     default void visit(ConnectByRootOperator connectByRootOperator) {

src/main/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapter.java

@@ -729,12 +729,40 @@ public <S> T visit(JsonAggregateFunction jsonAggregateFunction, S context) {
     @Override
     public <S> T visit(JsonFunction jsonFunction, S context) {
         ArrayList<Expression> subExpressions = new ArrayList<>();
+        for (JsonKeyValuePair keyValuePair : jsonFunction.getKeyValuePairs()) {
+            if (keyValuePair.getKey() instanceof Expression) {
+                subExpressions.add((Expression) keyValuePair.getKey());
+            }
+            if (keyValuePair.getValue() instanceof Expression) {
+                subExpressions.add((Expression) keyValuePair.getValue());
+            }
+        }
         for (JsonFunctionExpression expr : jsonFunction.getExpressions()) {
             subExpressions.add(expr.getExpression());
         }
+        if (jsonFunction.getInputExpression() != null) {
+            subExpressions.add(jsonFunction.getInputExpression().getExpression());
+        }
+        if (jsonFunction.getJsonPathExpression() != null) {
+            subExpressions.add(jsonFunction.getJsonPathExpression());
+        }
+        subExpressions.addAll(jsonFunction.getPassingExpressions());
+        if (jsonFunction.getOnEmptyBehavior() != null
+                && jsonFunction.getOnEmptyBehavior().getExpression() != null) {
+            subExpressions.add(jsonFunction.getOnEmptyBehavior().getExpression());
+        }
+        if (jsonFunction.getOnErrorBehavior() != null
+                && jsonFunction.getOnErrorBehavior().getExpression() != null) {
+            subExpressions.add(jsonFunction.getOnErrorBehavior().getExpression());
+        }
         return visitExpressions(jsonFunction, context, subExpressions);
     }
 
+    @Override
+    public <S> T visit(JsonTableFunction jsonTableFunction, S context) {
+        return visitExpressions(jsonTableFunction, context, jsonTableFunction.getAllExpressions());
+    }
+
     @Override
     public <S> T visit(ConnectByRootOperator connectByRootOperator, S context) {
         return connectByRootOperator.getColumn().accept(this, context);