Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions

On Sun, Jul 26, 2020 at 11:04 AM Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
>
> On Sat, Jul 25, 2020 at 5:08 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> >
> > On Fri, Jul 24, 2020 at 7:17 PM Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
> > >
> > > Your changes look fine to me. Additionally, I have changed a test
> > > case of getting the streaming changes in 0002. Instead of just
> > > showing the count, I am showing that the transaction is actually
> > > streaming.
> > >
> >
> > If you want to show the changes then there is no need to display 157
> > rows probably a few (10-15) should be sufficient. If we can do that
> > by increasing the size of the row then good, otherwise, I think it is
> > better to retain the test to display the count.
>
> I think in existing test cases also we are displaying multiple lines
> e.g. toast.out is showing 235 rows. But maybe I will try to reduce it
> to the less number of rows.

Changed, now only 27 rows.

--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com