On April 23, 2017 10:31:18 PM PDT, Petr Jelinek <petr.jelinek@2ndquadrant.com> wrote:
>On 24/04/17 04:31, Petr Jelinek wrote:
>So actually maybe running regression tests through it might be
>reasonable approach if we add new make target for it.
That sounds like a good plan.
>Note that the first patch is huge. That's because I needed to add
>alternative output for largeobject test because it uses fastpath
>function calls which are not allowed over replication protocol.
There's no need for that restriction, is there? At least for db walsenders...
>As part of this I realized that the parser fallback that I wrote
>originally for handling SQL in walsender is not robust enough as the
>lexer would fail on some valid SQL statements. So there is also second
>patch that does this using different approach which allows any SQL
>statement.
Haven't looked at the new thing yet, but I'm not particularly surprised... Wonder of there's a good way to fully
integrateboth grammars, without exploding keywords.
Andres
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.