Re: postgres_fdw: Emit message when batch_size is reduced - Mailing list pgsql-hackers

From Corey Huinker
Subject Re: postgres_fdw: Emit message when batch_size is reduced
Date
Msg-id CADkLM=fDPqgPQOBBBb-rhND-hCXSuujnvOvCRUaNjY=S4XvvkA@mail.gmail.com
Whole thread
Responses Re: postgres_fdw: Emit message when batch_size is reduced
List pgsql-hackers

- In the test case, choosing a batch_size > PQ_QUERY_PARAM_MAX_LIMIT seems a bit artificial, in that it's a value that can't work for any query with parameters, and someone might add sanity checks to batch_size in option.c, which would in turn invalidate the test case. A test case with a 2-column table should be able to get the same effect any batch_size greater than half of 65535, yes?




Thinking some more on this part, I think it would be worthwhile to check on assignments to batch_size, and issue a NOTICE or WARNING that the set value exceeds PQ_QUERY_PARAM_MAX_LIMIT, but does not reject the change because this value could theoretically (but very unlikely) get smaller in the future and we don't want to mess up any pg_restores or pg_upgrades.
 

pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: dict_synonym.c: fix truncation of multibyte sequence
Next
From: Jacob Champion
Date:
Subject: [PATCH] CI: Add a CPAN cache on Windows