postgres_fdw: Useless test in pgfdw_exec_cleanup_query_end() - Mailing list pgsql-hackers

From Etsuro Fujita
Subject postgres_fdw: Useless test in pgfdw_exec_cleanup_query_end()
Date
Msg-id CAPmGK14mm+=UjyjoWj_Hu7c+QqX-058RFfF+qOkcMZ_Nj52v-A@mail.gmail.com
Whole thread Raw
Responses Re: postgres_fdw: Useless test in pgfdw_exec_cleanup_query_end()
List pgsql-hackers
Another thing I noticed while working on [1] is $SUBJECT: this
function checks whether the given query string is non-NULL or not when
creating a WARNING message, but the function is always called with the
query string set, so it should be non-NULL.  I removed the check and
instead added an assertion ensuring that the query string is non-NULL.
(I added the assertion to pgfdw_exec_cleanup_query_begin() as well.)
Attached is a patch for that.

If there are no objections, I will apply the patch to HEAD only.

Best regards,
Etsuro Fujita

[1] https://www.postgresql.org/message-id/CAPmGK15DF6EE7O6hTLbe5-fHvPDwEx9vm-BOCN3dsKOjZCo7bw%40mail.gmail.com

Attachment

pgsql-hackers by date:

Previous
From: Ajin Cherian
Date:
Subject: Re: Introduce XID age and inactive timeout based replication slot invalidation
Next
From: Amit Kapila
Date:
Subject: Re: Introduce XID age and inactive timeout based replication slot invalidation