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 PostgreSQL #21

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

Conversation

kysrpex
Copy link

@kysrpex kysrpex commented Jul 4, 2024

Support loading mappings from PostgreSQL via the pg-native library. A major problem was that the proxy works watching files for changes (SQLite or JSON files). With PostgreSQL this is not possible, so I resorted to polling at regular intervals (configurable via the new --pollingInterval command line option) and PostgreSQL asynchronous notifications. See the new function watchPostgres and new section of the README for more details.

As a side effect, running the proxy requires nodejs v14 now. I have also included a package-lock.json file and version specifiers for axios, axios-retry, chai and mocha.

This feature is meant to work in conjunction with galaxyproject/galaxy#18481.

Pin project dependencies to a set known to work using package-lock.json.
Support loading mappings from PostgreSQL via the `pg-native` library.
@kysrpex
Copy link
Author

kysrpex commented Sep 5, 2024

galaxyproject/galaxy#18481 got merged, this PR would be the next step. From Git blame, it looks like the original maintainer was @jmchilton.

@jmchilton are you still a contact person for this repository? Who should I contact otherwise?

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