Re: [BUG] temporary file usage report with extended protocol and unnamed portals

Поиск
Список
Период
Сортировка
От Sami Imseih
Тема Re: [BUG] temporary file usage report with extended protocol and unnamed portals
Дата
Msg-id CAA5RZ0t==BMNxjg+aDT+5oSUKMb1ApLdv-jKWWNfZceHdbDwrA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [BUG] temporary file usage report with extended protocol and unnamed portals  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: [BUG] temporary file usage report with extended protocol and unnamed portals
Re: [BUG] temporary file usage report with extended protocol and unnamed portals
Список pgsql-hackers
> Just my 2c while looking at this particular part of the thread.  Now
> to the main patch proposed, v8 or v10..

I have been thinking about whether test coverage is worth it for temp
file logging. I think it is, but others may disagree. However, I also
don't think the current tests are correct.

For example, this is not true. We should now always log the correct query.
```
    ok(0, "The wrong query has been logged");
```

Also, the tests should be checking that we are logging "temporary file: "
before the next statement is logged.

I split up the actual fix and the corrected tests into separate patches.
They can be committed together if there is agreement that tests are worth it.

--
Sami

Вложения

В списке pgsql-hackers по дате отправления: