Thread: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
[pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
From
Akshay Joshi
Date:
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
Attachment
Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
From
Khushboo Vashi
Date:
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'
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 & RegardsAkshay JoshiSr. Software ArchitectEnterpriseDB Software India Private LimitedMobile: +91 976-788-8246
Re: [pgAdmin4][RM#5261] Support for Collation, FTS Templates, FTSDictionaries, FTS Configurations and FTS Parsers
From
Akshay Joshi
Date:
Thanks, Khushboo for review.
On Thu, Mar 19, 2020 at 1:25 PM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
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'
The issue has been fixed and committed.
Thanks,KhushbooOn 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 & RegardsAkshay JoshiSr. Software ArchitectEnterpriseDB Software India Private LimitedMobile: +91 976-788-8246
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246