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

Docker postgres #18

Merged
merged 3 commits into from
Nov 22, 2015
Merged

Docker postgres #18

merged 3 commits into from
Nov 22, 2015

Conversation

masinoa
Copy link

@masinoa masinoa commented Nov 2, 2015

This pull request (full details are in the buildmimic/postgres/README.md file):

  • provides necessary Dockerfile to run a container with a Postgres server
  • bash scripts on the Docker container automatically run the SQL scripts necessary to load the data from
    the csv data files to the Postgres database
  • minor changes to the SQL scripts to simplify CSV data loading and reduce error messages by using IF EXISTS clauses

@alistairewj alistairewj merged commit 52dbf32 into MIT-LCP:master Nov 22, 2015
@alistairewj
Copy link
Member

Sorry for the delay, have been quite busy on this one! Thanks for the code. Just tried to get it working on Windows and ran into a random assortment of wonderful Windows related errors. Still working through them. Wanted to check though - in the create tables script you omit the backslash in copy statements, i.e. you've written "COPY TABLE_NAME ...." instead of "\COPY TABLE_NAME ...". Is this intentional or a typo? At the moment I've changed it to \COPY.

@masinoa
Copy link
Author

masinoa commented Nov 22, 2015

@alistairewj Happy to contribute the code. The missing \ in the COPY statement was a typo. Everything ran OK in my tests so I guess I got "lucky" that either Postgres in general or Postgres on Linux was forgiving of this lapse.

@masinoa masinoa deleted the docker_postgres branch November 22, 2015 17:36
@tompollard tompollard mentioned this pull request Dec 24, 2015
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