On Wed, Sep 2, 2020 at 7:19 PM Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Wed, Sep 2, 2020 at 3:41 PM Dilip Kumar <dilipbal...@gmail.com> wrote: > > > > On Wed, Sep 2, 2020 at 10:55 AM Amit Kapila <amit.kapil...@gmail.com> > wrote: > > > > > > > > > > > > > We can combine the tests in 015_stream_simple.pl and > > > 020_stream_binary.pl as I can't see a good reason to keep them > > > separate. Then, I think we can keep only this part with the main patch > > > and extract other tests into a separate patch. Basically, we can > > > commit the basic tests with the main patch and then keep the advanced > > > tests separately. I am afraid that there are some tests that don't add > > > much value so we can review them separately. > > > > Fixed > > > > I have slightly adjusted this test and ran pgindent on the patch. I am > planning to push this tomorrow unless you have more comments. > Looks good to me. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com
