From: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Jacob Champion <pchampion(at)vmware(dot)com> |
Subject: | Re: Small issues with CREATE TABLE COMPRESSION |
Date: | 2021-05-08 14:49:03 |
Message-ID: | CAFiTN-twgPmohG7qj1HXhySq16h123y5OowsQR+5h1YeZE9fmQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, May 8, 2021 at 2:08 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Thu, May 06, 2021 at 09:33:53PM +0530, Dilip Kumar wrote:
> > On Thu, May 6, 2021 at 5:42 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> > >
> > > On Thu, May 06, 2021 at 10:45:53AM +0530, Dilip Kumar wrote:
> > > > Thanks, Robert and Michael for your input. I will try to understand
> > > > how it is done in the example shared by you and come up with the test
> > > > once I get time. I assume this is not something urgent.
> > >
> > > Thanks. FWIW, I'd rather see this gap closed asap, as features should
> > > come with proper tests IMO.
> >
> > I have done this please find the attached patch.
>
> No objections to take the approach to mark the lz4-related test with a
> special flag and skip them. I have three comments:
> - It would be good to document this new flag. See the comment block
> on top of %dump_test_schema_runs.
> - There should be a test for --no-toast-compression. You can add a
> new command in %pgdump_runs, then unmatch the expected output with the
> option.
> - I would add one test case with COMPRESSION pglz somewhere to check
> after the case of ALTER TABLE COMPRESSION commands not generated as
> this depends on default_toast_compression. A second test I'd add is a
> materialized view with a column switched to use lz4 as compression
> method with an extra ALTER command in create_sql.
I have fixed some of them, I could not write the test cases where we
have to ensure that 'ALTER TABLE COMPRESSION' is not appearing in the
dump. Basically, I don't have knowledge of the perl language so even
after trying for some time I could not write those 2 test cases. I
have fixed the remaining comments.
> > I don't have much idea about the MSVC script, but I will try to see
> > some other parameters and fix this.
>
> Thanks! I can dive into that if that's an issue. Let's make things
> compatible with what upstream provides, meaning that we should have
> some documentation pointing to the location of their deliverables,
> equally to what we do for the Perl and OpenSSL dependencies for
> example.
I have changed the documentation and also updated the Solution.pm. I
could not verify the windows build yet as I am not having windows
environment.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Add-pg_dump-test-case-for-TOAST-compression.patch | text/x-patch | 4.0 KB |
v2-0002-LZ4-option-for-windows-build-and-documentation-up.patch | text/x-patch | 2.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2021-05-08 15:04:01 | Re: [PATCH] Identify LWLocks in tracepoints |
Previous Message | Dilip Kumar | 2021-05-08 14:33:34 | Re: Small issues with CREATE TABLE COMPRESSION |