On Tue, Sep 1, 2020 at 8:33 PM Amit Kapila <amit.kapila16@gmail.com> wrote:>> On Tue, Sep 1, 2020 at 9:28 AM Amit Kapila <amit.kapila16@gmail.com> wrote:>> I have fixed all the comments except the below comments.> 1. verify the size of various tests to ensure that it is above> logical_decoding_work_mem.> 2. I have checked that in one of the previous patches, we have a test> v53-0004-Add-TAP-test-for-streaming-vs.-DDL which contains a test case> quite similar to what we have in> v55-0002-Add-support-for-streaming-to-built-in-logical-re/013_stream_subxact_ddl_abort.> If there is any difference that can cover more scenarios then can we> consider merging them into one test?>I have compared these two tests and found that the only thingadditional in the test case present inv53-0004-Add-TAP-test-for-streaming-vs.-DDL was that it was performingfew savepoints and DMLs after doing the first rollback to savepointand I included that in one of the existing tests in018_stream_subxact_abort.pl. I have added one test for Rollback,changed few messages, removed one test case which was not making anysense in the patch. See attached and let me know what you think aboutit?
pgsql-hackers by date:
Соглашаюсь с условиями обработки персональных данных
