Re: Case expression pushdown

Tom Lane писал 2021-07-26 18:18:
> Alexander Pyhalov <a.pyhalov@postgrespro.ru> writes:
>> [ 0001-Allow-pushing-CASE-expression-to-foreign-server-v6.patch ]
> 
> This doesn't compile cleanly:
> 
> deparse.c: In function 'foreign_expr_walker.isra.4':
> deparse.c:920:8: warning: 'collation' may be used uninitialized in
> this function [-Wmaybe-uninitialized]
>      if (collation != outer_cxt->collation)
>         ^
> deparse.c:914:3: warning: 'state' may be used uninitialized in this
> function [-Wmaybe-uninitialized]
>    switch (state)
>    ^~~~~~
> 
> These uninitialized variables very likely explain the fact that it 
> fails
> regression tests, both for me and for the cfbot.  Even if this weren't 
> an
> outright bug, we don't tolerate code that produces warnings on common
> compilers.
> 
>             regards, tom lane

Hi.

Of course, this is a patch issue. Don't understand how I overlooked 
this.
Rebased on master and fixed it. Tests are passing here (but they also 
passed for previous patch version).

What exact tests are failing?

-- 
Best regards,
Alexander Pyhalov,
Postgres Professional