|From:||Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>|
|To:||Michael Paquier <michael(dot)paquier(at)gmail(dot)com>|
|Cc:||Piotr Stefaniak <email(at)piotr-stefaniak(dot)me>, "obartunov(at)gmail(dot)com" <obartunov(at)gmail(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>, Teodor Sigaev <teodor(at)postgrespro(dot)ru>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, andrew Dunstan <andrew(at)dunslane(dot)net>|
|Subject:||Re: [HACKERS] SQL/JSON in PostgreSQL|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 04.11.2017 01:52, Nikita Glukhov wrote:
>> On 03.11.2017 15:07, Michael Paquier wrote:
>> About the patch set, I had a look at the first patch which is not that
>> heavy, however it provides zero documentation, close to zero comments,
>> but adds more than 500 lines of code. I find that a bit hard to give
>> an opinion on, having commit messages associated to each patch would
>> be also nice. This way, reviewers can figure what's going out in this
>> mess and provide feedback.
> Sorry that comments and commit messages are still absent. I am going
> to do it
> in the next version of these patches where SQL/JSON constructors
> will be fixed.
Attached the new version of the patches where displaying of SQL/JSON
constructor nodes was fixed. I decided not to invent new nodes but to
existing FuncExpr, Aggref, WindowFunc nodes with new formatting fields that
give us ability to override default displaying in ruleutils.c. Also new
invisible CoercionForm was added for hiding casts in which FORMAT and
clauses are transformed.
>> Making things incremental is welcome as
>> well, for example in the first patch I have a hard way finding out why
>> timestamps are touched to begin with.
> Timestamp's code was touched to add support of two features needed for
> .datetime() item method by standard:
> - TZH and TZM template patterns
> - datetime components recognition
> I absolutely agree that this should be in a separate patch.
TZH and TZM template patterns were moved into a separate patch.
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
|Next Message||Amit Langote||2017-11-15 02:16:51||Re: [Sender Address Forgery]Re: [HACKERS] path toward faster partition pruning|
|Previous Message||Michael Paquier||2017-11-15 01:05:49||Re: [HACKERS] Assertion failure when the non-exclusive pg_stop_backup aborted.|