Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
Date
Msg-id CAFOhELdLxRaQ4gULv8KHen3CP9c2M1PPyPXX-S8Eba4b0UBK=Q@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List 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 by date:

Previous
From: Aditya Toshniwal
Date:
Subject: Re: [pgAdmin4][RM 5271] To change the switch control colors
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Added support of Collation, FTS Configuration, FTS Di