Skip to content
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

Add support for GeoPackage to deegree #1437

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

tfr42
Copy link
Member

@tfr42 tfr42 commented Dec 8, 2022

With this PR the support of GeoPackage is added to deegree. This PR is an update of the source PR #333 (developed by @diegomigliavacca) with changes to be able to merge into the 3.5 main branch.
It has been tested with the workspace available here: https://github.com/lat-lon/deegree-workspaces/tree/gpkgWorkspace-8209

When merging this PR consider to make a squash and merge commit.

@tfr42 tfr42 added this to the 3.5 milestone Dec 8, 2022
@tfr42 tfr42 added WFS deegree Web Feature Service WMS deegree Web Map Service WMTS deegree Web Map Tile Service feature feature in progress console deegree administration console schema change change to deegree configuration files labels Dec 8, 2022
@tfr42
Copy link
Member Author

tfr42 commented Dec 21, 2022

Proposal is to rename the dialect to SqliteDialect instead of GpkgDialect.

@tfr42 tfr42 modified the milestones: 3.5, 3.6 Dec 21, 2022
@stephanr
Copy link
Member

As discussed in the tmc meeting, the documentation should mention the native reference to the native dependencies.
See also https://github.com/xerial/sqlite-jdbc#how-does-sqlitejdbc-work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
console deegree administration console feature feature in progress schema change change to deegree configuration files WFS deegree Web Feature Service WMS deegree Web Map Service WMTS deegree Web Map Tile Service
Projects
Open activities for TMC
  
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

7 participants