On 2017/12/02 2:57, Robert Haas wrote:
> On Fri, Dec 1, 2017 at 2:44 AM, Amit Langote
> <Langote_Amit_f8@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
