Skip to content

MySQL OAF provider #2004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

C-Loftus
Copy link
Contributor

Overview

  • Adds a MySQL OAF provider with associated tests
    • MySQL is very similar to postgis but doesn't support the same geospatial functions so it needs some different underlying behavior
  • This PR keeps all existing postgres functionality; in doing so it puts all core sql functionality into a base class that both postgres and mysql providers can inherit from; (this could be used for other sql providers in the future if desired)
  • Fixed formatting in existing postgis provider to follow the rest of the project (single quotes etc)

Related Issue / discussion

N/A

Additional information

N/A

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

C-Loftus and others added 2 commits April 24, 2025 15:53
- mysql provider and associated tests
- refactor postgres and mysql to use a shared base class that they both inherit from
- handle geometry / encoding necessary to use mysql wity sqlalchemy

---------

Co-authored-by: Benjamin Webb <40066515+webb-ben@users.noreply.github.com>
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.

1 participant