Re: [pgAdmin][RM5398] Bad display of tables storage parameters in the "SQL" tab

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin][RM5398] Bad display of tables storage parameters in the "SQL" tab
Дата
Msg-id CANxoLDe43r8Dvn1n=GxhPKU-gPVWjV+_+2FgdMAky3wwDmjfcA@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin][RM5398] Bad display of tables storage parameters in the "SQL" tab  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Ответы Re: [pgAdmin][RM5398] Bad display of tables storage parameters in the "SQL" tab  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Список pgadmin-hackers
Hi Aditya

RESQL test cases are failing for all database server and one minor issue I found:
  • open the properties dialog for any existing table and set the "Custom auto-vacuum?" to Yes.
  • Save button gets enabled and when clicking on it we get "can't execute an empty query" error.

Please fix the above and send the patch again. 

On Tue, May 5, 2020 at 12:07 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to:
1) Fix the SQL generated for autovacuum options. It was not displaying the vacuum options until autovacuum_enabled is set to true. Vacuum options setting does not depend on autovacuum_enabled, and this was being wrongly interpreted by pgAdmin.
2) pgAdmin allows only true or false values for autovacuum_enabled, and there is no way to reset it individually without disabling custom autovacuum. To fix this, the control is changed from switch to three state buttons. Screenshot below:
Screenshot 2020-05-05 at 11.44.11 AM.png
3) Setting any vacuum options to zero made the option disappear from the SQL.
4) Right click on a table with vacuum options set and create a new table. The parameters tab will have the options prefilled. This is fixed.
5) Changed the UI a bit to remove unnecessary space around. Added "Autovacuum" before enabled.
6) The SQL generated was appending a ".0" to decimal vacuum options. Changed to remove the leading ".0".
7) Test cases updated to increase the scenarios.
8) Documentation update.

Please review.

--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246
Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: [pgAdmin][RM5480]Background job creation failure.
Следующее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin][RM5398] Bad display of tables storage parameters in the "SQL" tab