Skip to content

[feat] Support for JSON_TABLE #2328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
manticore-projects merged 13 commits into from
Apr 2, 2026
Merged

[feat] Support for JSON_TABLE #2328

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
0b85753
[feat] Initial support for JSON_TABLE
ANeumann82 Oct 13, 2025
8773eea
[feat] Additional support for JSON_TABLE
ANeumann82 Oct 15, 2025
0c55268
[chore] Code Review
ANeumann82 Oct 17, 2025
cb1654a
Merge branch 'master' into json_table_support
ANeumann82 Nov 25, 2025
81ccfa6
Merge branch 'master' into json_table_support
ANeumann82 Mar 15, 2026
d39df7f
[chore] Started to add oracle JSON_TABLE features
ANeumann82 Mar 15, 2026
3908529
[feat] Fixed onError for Oracle, added onEmptyClause, addedParsingTyp…
ANeumann82 Mar 23, 2026
5775a78
[feat] Added EXISTS keyword
ANeumann82 Apr 1, 2026
96c40e1
[chore] spotless
ANeumann82 Apr 1, 2026
8d3b1d6
[chore] cleanup
ANeumann82 Apr 1, 2026
ee3e4f4
Merge branch 'refs/heads/master' into json_table_support
ANeumann82 Apr 1, 2026
3f5edea
[bugfix] Fixed token limit again...
ANeumann82 Apr 1, 2026
c65de44
[chore] One more test, code cleanup
ANeumann82 Apr 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[bugfix] Fixed token limit again...
  • Loading branch information
@ANeumann82
ANeumann82 committed Apr 1, 2026
commit 3f5edeaf325614596b5a9b5fade16b71b7636834
28 changes: 18 additions & 10 deletions src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -805,7 +805,6 @@ String NonReservedWord() :
| tk=<K_ALGORITHM: "ALGORITHM">
| tk=<K_ALIGN:"ALIGN">
| tk=<K_ALTER:"ALTER">
| tk=<K_ALLOW:"ALLOW">
| tk=<K_ALWAYS:"ALWAYS">
| tk=<K_ANALYZE:"ANALYZE">
| tk=<K_APPEND_ONLY:"APPEND_ONLY">
Expand Down Expand Up @@ -878,7 +877,6 @@ String NonReservedWord() :
| tk=<K_DESC:"DESC">
| tk=<K_DESCRIBE:"DESCRIBE">
| tk=<K_DISABLE : "DISABLE">
| tk=<K_DISALLOW:"DISALLOW">
| tk=<K_DISCARD : "DISCARD">
| tk=<K_DISCONNECT:"DISCONNECT">
| tk=<K_DIV:"DIV">
Expand Down Expand Up @@ -961,7 +959,6 @@ String NonReservedWord() :
| tk=<K_KILL:"KILL">
| tk=<K_FN:"FN">
| tk=<K_LAST: "LAST">
| tk=<K_LAX: "LAX">
| tk=<K_LEADING:"LEADING">
| tk=<K_LESS:"LESS">
| tk=<K_LEVEL:"LEVEL">
Expand Down Expand Up @@ -1350,7 +1347,7 @@ TOKEN : /* Data Types */
| <#TYPE_UHUGEINT: "UHUGEINT" >
| <#TYPE_REAL: "REAL" | "FLOAT4" | "FLOAT">
| <#TYPE_DOUBLE: "DOUBLE" | "PRECISION" | "FLOAT8" | "FLOAT64">
| <#TYPE_VARCHAR: "NVARCHAR" | "VARCHAR" | "NCHAR" | <K_CHAR> | "BPCHAR" | "TEXT" | "STRING" | <K_CHARACTER> | "VARYING" | "VARCHAR2">
| <#TYPE_VARCHAR: "NVARCHAR" | "VARCHAR" | "NCHAR" | <K_CHAR> | "BPCHAR" | "TEXT" | "STRING" | <K_CHARACTER> | "VARYING">
| <#TYPE_TIME: "TIMETZ" >
| <#TYPE_TIMESTAMP: "TIMESTAMP_NS" | "TIMESTAMP_MS" | "TIMESTAMP_S" >

Expand Down Expand Up @@ -5861,8 +5858,6 @@ FromItem FromItem() #FromItem:
|| (getToken(1).kind == K_LATERAL && getToken(2).kind != OPENING_BRACKET) })
fromItem=TableFunction()
|
// fromItem=JsonTable()
// |
LOOKAHEAD(3) fromItem=Table()
|
LOOKAHEAD(ParenthesedFromItem()) fromItem = ParenthesedFromItem()
Expand Down Expand Up @@ -9684,16 +9679,28 @@ JsonTableFunction.JsonTableColumnDefinition JsonTableColumnDefinition() : {
(
<K_FOR> <K_ORDINALITY> { valueColumnDefinition.setForOrdinality(true); }
|
[ dataType = ColDataType() { valueColumnDefinition.setDataType(dataType); } ]
[
// Very ugly: ColDataType can consume an IDENTIFIER, which is fine, but we don't want it to
// consume an ALLOW or DISALLOW because that's a keyword for Oracle in this place.
// So we make a LOOKAHEAD on ColDataType, BUT we exclude the two cases for the IDENTIFIER
LOOKAHEAD(
Comment thread
manticore-projects marked this conversation as resolved.
ColDataType(),
{ !(getToken(1).kind == S_IDENTIFIER && (
getToken(1).image.equalsIgnoreCase("ALLOW")
|| getToken(1).image.equalsIgnoreCase("DISALLOW"))) } )
dataType = ColDataType() { valueColumnDefinition.setDataType(dataType); }
]
[
<K_FORMAT> <K_JSON> { valueColumnDefinition.setFormatJson(true); }
[ <K_ENCODING> encoding = JsonEncoding() { valueColumnDefinition.setEncoding(encoding); } ]
]
[
(
<K_ALLOW> { valueColumnDefinition.setScalarsType(JsonFunction.ScalarsType.ALLOW); }
LOOKAHEAD({ getToken(1).kind == S_IDENTIFIER && getToken(1).image.equalsIgnoreCase("ALLOW") })
JsonKeyword("ALLOW") { valueColumnDefinition.setScalarsType(JsonFunction.ScalarsType.ALLOW); }
|
<K_DISALLOW> { valueColumnDefinition.setScalarsType(JsonFunction.ScalarsType.DISALLOW); }
LOOKAHEAD({ getToken(1).kind == S_IDENTIFIER && getToken(1).image.equalsIgnoreCase("DISALLOW") })
JsonKeyword("DISALLOW") { valueColumnDefinition.setScalarsType(JsonFunction.ScalarsType.DISALLOW); }
)
JsonKeyword("SCALARS")
]
Expand Down Expand Up @@ -9903,7 +9910,8 @@ JsonTableFunction.JsonTableParsingTypeClause JsonTableParsingTypeClause() : {
(
<K_STRICT> { parsingType.setType(JsonTableFunction.JsonTableParsingType.STRICT); }
|
<K_LAX> { parsingType.setType(JsonTableFunction.JsonTableParsingType.LAX); }
LOOKAHEAD({ getToken(1).kind == S_IDENTIFIER && getToken(1).image.equalsIgnoreCase("LAX") })
JsonKeyword("LAX") { parsingType.setType(JsonTableFunction.JsonTableParsingType.LAX); }
)
<CLOSING_BRACKET>
{
Expand Down
6 changes: 3 additions & 3 deletions src/test/java/net/sf/jsqlparser/expression/JsonTableOracleTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,9 @@ void testExistsColumns(String jsonTableStr) throws JSQLParserException {
"JSON_TABLE(document COLUMNS( val BLOB PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val JSON PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VECTOR PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR2 PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR2(240) PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR2(240) FORMAT JSON PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR(240) PATH '$.pathTest'))",
"JSON_TABLE(document COLUMNS( val VARCHAR(240) FORMAT JSON PATH '$.pathTest'))",

// These would require adapting ColDataType in Line 10176
// "JSON_TABLE(document COLUMNS( val VARCHAR2(500 BYTE) PATH '$.pathTest'))",
Expand Down
Loading