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

Develop a new major version #290

Open
amercader opened this issue Aug 22, 2022 · 1 comment
Open

Develop a new major version #290

amercader opened this issue Aug 22, 2022 · 1 comment

Comments

@amercader
Copy link
Member

amercader commented Aug 22, 2022

ckanext-spatial is an old extension and hasn't had a big refactoring in a long while. Postgis and other python requirements make it difficult to install and deploy. In addition, the spatial search capabilities in Solr have evolved significantly since the current implementation and both indexing and querying can be greatly simplified in ckanext-spatial.

The main areas of work will be:

  • Dropping PostGIS requirement (Don't require PostGIS unless using it as a search backend #195 ). Support only Solr-based search (the PostGIS one is not suitable for production anyway)
  • Refactor search logic. Use modern Solr fields to support the existing backend modes (solr for bboxes and solr-spatial-field for polygons, points, etc), they can be probably merged into a single implementation. Write actual Solr based tests. See Refactor spatial search to use modern Solr options #291
    This work could be tagged as v2.0.0, and ideally will only target CKAN 2.9+ in Python 3, although we can re-assess this when we know the scope of the changes

cc @avdata99

@avdata99
Copy link
Member

@amercader I think #289 could be part of this mayor change.
I assume you want to take care of this, but let me know if I have to do any of the work. Maybe start building version 2.0.0 for Python 3.

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

No branches or pull requests

2 participants