On 01/10/2018 05:42 PM, Nikita Glukhov wrote: > Attached new 8th version of jsonpath related patches. Complete > documentation is still missing. > > The first 4 small patches are necessary datetime handling in jsonpath: > 1. simple refactoring, extracted function that will be used later in > jsonpath > 2. throw an error when the input or format string contains trailing > elements > 3. avoid unnecessary cstring to text conversions > 4. add function for automatic datetime type recognition by the > presence of formatting components > > Should they be posted in a separate thread? >
The first of these refactors the json/jsonb timestamp formatting into a single function, removing a lot of code duplication. The involves exposing time2tm() and timetz2tm(). I don't think that's a tragedy, so unless there is any objection I propose to commit it shortly. The next three expose a bit more of the date/time API. I'm still reviewing those. cheers andrew -- Andrew Dunstan https://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
