presto icon indicating copy to clipboard operation
presto copied to clipboard

Add support for geometry and geography in postgres connector data typ…

Open Mariamalmesfer opened this issue 6 months ago • 3 comments

Added support for the GEOMETRY type in postgres connector.

Description

Motivation and Context

To enable reading geometry data from postgres tables using Presto.

Impact

Test Plan

Contributor checklist

  • [ ] Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • [ ] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • [ ] Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • [ ] If release notes are required, they follow the release notes guidelines.
  • [ ] Adequate tests were added if applicable.
  • [ ] CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

Postgres Connector Changes
* Add support for `GEOMETRY <https://prestodb.io/docs/current/language/types.html#geospatial>`_ type in the Postgres connector.

Mariamalmesfer avatar Jun 03 '25 12:06 Mariamalmesfer

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: Mariamalmesfer / name: Mariam AlMesfer (4d15916d07430efce42b97cd26d5d6dd7a3fb4d0)

Will https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/postgresql.rst#type-mapping need to be updated?

steveburnett avatar Jun 03 '25 14:06 steveburnett

Thanks for the release note entry! A couple of suggestions.

== RELEASE NOTES ==

MySQL Connector Changes
* Add support for `GEOMETRY <https://prestodb.io/docs/current/language/types.html#geospatial>`_ type in the Postgres connector.

steveburnett avatar Jun 18 '25 13:06 steveburnett

Please resolve the conflicts.

pratyakshsharma avatar Jul 15 '25 11:07 pratyakshsharma

Please resolve the conflicts.

Resolved the conflicts.

Mariamalmesfer avatar Jul 15 '25 13:07 Mariamalmesfer

The commit message can be simplified to Add read support for geometry and geography data types in postgres connector

pratyakshsharma avatar Jul 16 '25 12:07 pratyakshsharma

@prestodb/committers can someone please review this?

Mariamalmesfer avatar Aug 03 '25 17:08 Mariamalmesfer