On Fri, Sep 15, 2023 at 10:13:10AM +0200, Daniel Gustafsson wrote: >> On 15 Sep 2023, at 04:39, Kyotaro Horiguchi <horikyota....@gmail.com> wrote: >> It seems to work fine. However, if we're aiming for consistent >> spacing, the "IS NULL" (two spaces in between) might be an concern. > > I don't think that's a problem. I would rather have readable C code and two > spaces in the generated SQL than contorting the C code to produce less > whitespace in a query few will read in its generated form.
I think we could pretty easily avoid the extra space and keep the C code relatively readable. These sorts of things bug me, too (see 2af3336). -- Nathan Bossart Amazon Web Services: https://aws.amazon.com