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 #443

Closed
wants to merge 2 commits into from
Closed

Add support for PostgreSQL #443

wants to merge 2 commits into from

Conversation

jkutner
Copy link

@jkutner jkutner commented Dec 17, 2015

A few concerns:

  • Search triggers and table are not included in postgresql schema. I couldn't get search working, but I think that's because Lucene.
  • Environment variables now override default and config.clj for uberjar (couldn't figure out how to set PG URL without hard coding otherwise, but this is generally good i think).
  • Depending on how the JDBC URL is set, the file clojars-postgres.sql or clojars-postgresql.sql might be used. I'll fix this in a future PR.

@jkutner jkutner mentioned this pull request Dec 17, 2015
@tobias
Copy link
Member

tobias commented Dec 30, 2015

Thanks for taking a look at this - I hope to have a chance to review and merge it soon.

@tobias
Copy link
Member

tobias commented Dec 31, 2019

Thanks again for this @jkutner - I know you did it years ago, but I used it as a starting point for finally moving us to postgres in #736.

@tobias tobias closed this Dec 31, 2019
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.

None yet

2 participants