Skip to content

[FLINK-XXXXX][table] Add GEOGRAPHY logical type model#28679

Draft
davidchaava wants to merge 2 commits into
apache:masterfrom
akvelon:task/add-geography-logical-type-model
Draft

[FLINK-XXXXX][table] Add GEOGRAPHY logical type model#28679
davidchaava wants to merge 2 commits into
apache:masterfrom
akvelon:task/add-geography-logical-type-model

Conversation

@davidchaava

Copy link
Copy Markdown

What is the purpose of the change

This draft PR adds the initial GEOGRAPHY logical type model for Flink.

It is intended to support the ongoing FLIP discussion for native GEOGRAPHY
support in Flink SQL / Table API and to make the proposed Flink-side type shape
easier to review.

This PR is not intended to be merged before the FLIP direction is agreed.

Brief change log

  • Add LogicalTypeRoot.GEOGRAPHY.
  • Add GeographyType.
  • Extend logical type visitors for GEOGRAPHY.
  • Extend logical type parsing for the GEOGRAPHY serializable string.
  • Add logical type tests.

Verifying this change

This change adds tests for the new logical type model:

  • LogicalTypesTest
  • LogicalTypeParserTest

Does this pull request potentially affect one of the following parts:

  • Dependencies: no
  • Public API: no
  • Serializers: no
  • Runtime per-record code paths: no
  • Deployment or recovery: no
  • S3 file system connector: no

Documentation

  • Does this pull request introduce a new feature? yes
  • If yes, how is the feature documented? not documented yet; this is a draft PR for the FLIP discussion

@flinkbot

flinkbot commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants