Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab. - Mailing list pgadmin-hackers
From | Dave Page |
---|---|
Subject | Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab. |
Date | |
Msg-id | CA+OCxoysCoVVJEqNx7ZoYSpMQ_FYrY5+EA+GHPeHT2+wT87orw@mail.gmail.com Whole thread Raw |
In response to | Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab. (Dave Page <dpage@pgadmin.org>) |
Responses |
Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.
|
List | pgadmin-hackers |
Hi Khushboo,
Looks like this patch broke the regressions tests :-(. Can you take a look ASAP please?
Thanks!
2017-06-12 10:04:11,938: INFO werkzeug: 127.0.0.1 - - [12/Jun/2017 10:04:11] "GET /browser/table/sql/1/1/12669/ 2200/81920 HTTP/1.1" 500 -
Traceback (most recent call last): File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 2000, in __call__
return self.wsgi_app(environ, start_response)
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1991, in wsgi_app
response = self.make_response(self. handle_exception(e))
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1567, in handle_exception
reraise(exc_type, exc_value, tb)
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1988, in wsgi_app
response = self.full_dispatch_request()
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1641, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1544, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1639, in full_dispatch_request
rv = self.dispatch_request()
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/app.py", line 1625, in dispatch_request
return self.view_functions[rule. endpoint](**req.view_args)
File "/Users/pivotal/.pyenv/ versions/2.7.10/envs/pgadmin/ lib/python2.7/site-packages/ flask/views.py", line 84, in view
return self.dispatch_request(*args, **kwargs)
File "/Users/pivotal/workspace/ pgadmin4/web/pgadmin/browser/ utils.py", line 235, in dispatch_request
return method(*args, **kwargs)
File "/Users/pivotal/workspace/ pgadmin4/web/pgadmin/browser/ server_groups/servers/ databases/schemas/tables/__ init__.py", line 315, in wrap
return f(*args, **kwargs)
File "/Users/pivotal/workspace/ pgadmin4/web/pgadmin/browser/ server_groups/servers/ databases/schemas/tables/__ init__.py", line 2555, in sql
data = self._formatter(did, scid, tid, data)
File "/Users/pivotal/workspace/ pgadmin4/web/pgadmin/browser/ server_groups/servers/ databases/schemas/tables/__ init__.py", line 1081, in _formatter
data = self._columns_formatter(tid, data)
File "/Users/pivotal/workspace/ pgadmin4/web/pgadmin/browser/ server_groups/servers/ databases/schemas/tables/__ init__.py", line 663, in _columns_formatter
column['attlen'] = matchObj.group(1)
AttributeError: 'NoneType' object has no attribute 'group'
On Mon, Jun 12, 2017 at 12:49 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks, applied.On Fri, Jun 9, 2017 at 10:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached updated patch.Fixed the PostGIS Datatypes in SQL tab, Create / Update dialogues for Table, Column, Foreign Table and Type node.Regarding the review comment given by Dave (Also, I there should be a space between the , and value in the_geom geometry(MultiPolygon,3310).), won't be fixed as this format is returned by format_type() function itself.As per the conversation with Dave, I have logged RM #2473 to add support for PostGIS Datatypes.Thanks,KhushbooOn Tue, Jun 6, 2017 at 6:49 PM, Dave Page <dpage@pgadmin.org> wrote:On Tue, Jun 6, 2017 at 2:15 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: On 6 Jun 2017 18:36, "Dave Page" <dpage@pgadmin.org> wrote:On Tue, Jun 6, 2017 at 1:23 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: On 6 Jun 2017 17:42, "Dave Page" <dpage@pgadmin.org> wrote:HiThe table and column dialogues seem to have related problems. For example, the table dialogue shows the length of the geometry field in the test case as 847,380, whilst the column dialogue shows no length at all.What should be there in length and precision fields in this case?I have no idea :-). What did pgAdmin 3 do here? I assume there's some way to decode the typmod into the desired values, and map them back to appropriate SQL.I will try to figure out the way.pgAdmin 3 has the same issue, so I have asked for the suggestion.Oh, OK - thanks. Maybe we need to reuse the length and precision fields for these types. That is, I believe, essentially how PostGIS passes extra info to the column definitions.Also, I there should be a space between the , and value in the_geom geometry(MultiPolygon,3310).Thanks.--On Mon, Jun 5, 2017 at 8:32 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Fixed PostGIS datatype on SQL tab for Table, column, Foreign Table and Type node.Please find the attached updated patch.Thanks,KhushbooOn Thu, Jun 1, 2017 at 6:36 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: On Thu, Jun 1, 2017 at 5:27 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,On Thu, Jun 1, 2017 at 4:57 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Khushboo,Please include Type node and Foreign Table node in your fix :-)I don't think Type node requires this change.Yes, It do require because user can create composite type from existing types.Foreign table changes added in the attached updated patch.--Thanks Murtuza for pointing this out.Regards,On Thu, Jun 1, 2017 at 3:23 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: --Hi,Please find the attached patch to fix RM #2324 : PostGIS datatypes not showing up properly on SQL tab.Thanks,Khushboo
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment
pgadmin-hackers by date: