On 2017/12/02 2:57, Robert Haas wrote: > On Fri, Dec 1, 2017 at 2:44 AM, Amit Langote > <langote_amit...@lab.ntt.co.jp> wrote: >> I forgot to consider the fact that mtstate could be NULL in >> ExecSetupPartitionTupleRouting(), so would result in dereferencing NULL >> pointer when called from CopyFrom(), which fixed in the attached updated >> patch. > > a ? b : false can more simply be spelled a && b. > > Committed after changing it like that, fixing the broken documentation > build, and making minor edits to the comments and documentation.
Thanks for committing. Regards, Amit
