On Fri, Dec 13, 2024 at 12:27:37PM +0900, Sutou Kouhei wrote:
> Oh, sorry... I attached wrong patch...
> I attach the v4 patch that includes this case.
Sounds fair to me as a beginning for the code paths setting
need_transcoding.
Note that using "test" as table name for the tests is not a good idea,
as this could very likely conflict with some concurrent activity. I
would also add two RESET queries to remove the dependency to
client_encoding once the test has no need to rely on it. No need to
send a new patch for all that, just noticing in passing.
--
Michael