Re: [pgAdmin4][Patch]: Foreign Data Wrapper

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin4][Patch]: Foreign Data Wrapper
Дата
Msg-id CANxoLDcBLo+3seFN24t9RcUJDwYMJksLxZq4EvxP2vHr-6YQGw@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][Patch]: Foreign Data Wrapper  (Neel Patel <neel.patel@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch]: Foreign Data Wrapper  (Neel Patel <neel.patel@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi Neel 

Below are my review comments
  • For the consistency order in the privileges control should be Grantee, Privileges and Granter.
  • Correct the SQL from "DROP Foreign Data Wrapper" to "DROP FOREIGN DATA WRAPPER" in sql pane.
  • For Owner control there should a validation like "Owner can't be empty" when user tries to modify owner for Foreign data wrapper, Foreign Servers.
  • Correct the SQL from "DROP Foreign Server" to "DROP SERVER" in sql pane when click on any Foreign Server node.
  • When click on database node it should show menu for "Create-> Foreign Data Wrapper", when click on any Foreign data wrapper node it should show menu for "Create-> Foreign Data Wrapper" and "Create-> Foreign Server", when click on any Foreign Server node it should show menu for "Create-> Foreign Server" and "Create-> User Mapping".
  • Correct the SQL from "DROP User Mapping enterprisedb" to "DROP USER MAPPING FOR enterprisedb SERVER fs1" in sql pane when click on any User Mapping node.
  • On postgresql 9.1 and 9.2 Foreign Data wrapper node failed with below error when clicked on properties or sql tab.
       function expression in FROM cannot refer to other relations of same query level LINE 15: aclexplode(fdwacl) d ^
Apart from the above please fixed warning as much as possible using pep8 tool.

On Fri, Feb 5, 2016 at 5:29 PM, Neel Patel <neel.patel@enterprisedb.com> wrote:
Hi All,

Please find attached patch file that include the below three nodes under the database node.

    - Foreign Data Wrappers
    - Foreign Servers
    - User Mappings

Do review it and let me know for any comments.

Thanks,
Neel Patel


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers




--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

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

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: [pgAdmin4] [Patch]: Language Module
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Fix the display of elapsed query time for queries o