>
>
>>> - 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.