Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
Дата
Msg-id CAFOhELdLxRaQ4gULv8KHen3CP9c2M1PPyPXX-S8Eba4b0UBK=Q@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Hi Akshay,

The patch looks good to me.

Apart from this patch, I encountered below error.

"/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/schema_diff_utils.py", line 271, in get_sql_from_submodule_diff
   parent_target_data=target
 File "/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/partitions/__init__.py", line 790, in ddl_compare
   target_data=parent_target_data)
 File "/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/utils.py", line 142, in wrap
   return f(*args, **kwargs)
 File "/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/partitions/__init__.py", line 508, in get_sql_from_diff
   schema_diff=True)
 File "/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/utils.py", line 1369, in get_partitions_sql
   part_data['partition_scheme'] = self.get_partition_scheme(row)
 File "/Users/khushboovashi/Projects/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/utils.py", line 695, in get_partition_scheme
   for row in data[part_keys]:
KeyError: 'partition_keys'


Thanks,
Khushboo

On Wed, Mar 18, 2020 at 4:28 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Hackers, 

Attached is the patch to add support for Collation, FTS Templates, FTS Dictionaries, FTS Configurations and FTS Parsers in Schema Diff Tool.

The patch contains some minor fix where parameters are added in the difference list even though it's size is zero. I have also added the test cases.

Please review it.

--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

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

Предыдущее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin4][RM 5271] To change the switch control colors
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Added support of Collation, FTS Configuration, FTS Di