Uh oh!
There was an error while loading. Please reload this page.
Add support for MSSQL's JSON_ARRAY/JSON_OBJECT expr - #1507
Conversation
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
I modified existing DECLARE @id_key nvarchar(10) = N'id',@id_value nvarchar(64) = NEWID();
SELECT JSON_OBJECT(LOWER('USER_NAME') : USER_NAME(), LOWER(@id_key) : @id_value, 'sid':(SELECT @@SPID));Break change: pubenumFunctionArg{Named{name:Expr,// <<arg:FunctionArgExpr,operator:FunctionArgOperator,},Unnamed(FunctionArgExpr),}Please take a look! |
iffyio
left a comment
There was a problem hiding this comment.
Thanks @gaoqiangz I think this is looking good! Left some comments
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
gaoqiangz
commented
Nov 13, 2024
I refactored function parameter list parsing,add some |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
Thanks again @gaoqiangz and @iffyio and @lovasoa |
This just updates the postgres dialect definition to add support for the json_object syntax added in apache#1507 The syntax was initially added for mssql but also works in postgres See https://www.postgresql.org/docs/current/functions-json.html See apache#1545
lovasoa
commented
Nov 24, 2024
I opened a new PR to enable this syntax in postgres, which supports it too: #1546 |
Solves #1506