Re: [pgAdmin4][Patch]: RM #4110 "Updating 'Custom auto-vacuum?' property throws error for Materialized View."

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM #4110 "Updating 'Custom auto-vacuum?' property throws error for Materialized View."
Date: 2019-03-29 14:11:10
Message-ID: CA+OCxox8ZfZ5ktapzMsMSFcXHmYtT6ZCeJC7y0+epB6V-u0D-A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Thu, Mar 28, 2019 at 1:15 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Hackers,
>
> I have started working on RM #4110 "Updating 'Custom auto-vacuum?'
> property throws error for Materialized View." and while working I have
> figured out following issues:
>
> - Reset some/all the parameter for table and toast table is not
> implemented.
> - Value of "Custom auto-vacuum?" is not retrieved and set properly.
> - When user set the "Custom auto-vacuum?" to "Yes" and none of the
> parameter is set previously then set the value of "Enabled?" control to
> "Yes".
> - Alignment of Table and remove extra margin from "Parameters" tab.
>
> Attached is the patch to fix all the above issues. Added API test cases.
> Please review it.
>

I set the autovacuum_enabled=true and autovacuum_analyze_scale_factor=1 on
the test mview, but only the enabled setting is reflected when I re-open
the properties dialogue. Other settings seems to be fine - it's just
autovacuum_analyze_scale_factor that's broken.

It's also not shown in the reverse-engineered SQL - neither is
autovacuum_enabled.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2019-03-29 14:20:53 Re: [pgAdmin4][Patch]: RM #4110 "Updating 'Custom auto-vacuum?' property throws error for Materialized View."
Previous Message Usman Muzaffar 2019-03-29 14:10:25 Feature Tests query tool: unnecessary print statement removal