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

defined type for creating database schemas #456

Merged
merged 1 commit into from
Jul 29, 2014
Merged

defined type for creating database schemas #456

merged 1 commit into from
Jul 29, 2014

Conversation

igalic
Copy link
Contributor

@igalic igalic commented Jul 25, 2014

This defined type helps create database schemas, and assign them to an
owner. It is closely modeled after Postgresql::Server::Tablespace.
It uses PostgreSQL's builtin IF NOT EXISTS to guarantee idempotency.
(>= 9.3, else it checks pg_namespace).

n.b.: This defined type requires that a db is passed. This is a
concious design decision, since we find it rather useless to create such
schemas in the default postgres database, and if were useful, one
can always "over-specify".

This addresses MODULES-1098.

This defined type helps create database schemas, and assign them to an
`owner`. It is closely modeled after Postgresql::Server::Tablespace.
It uses PostgreSQL's builtin IF NOT EXISTS to guarantee idempotency.
(>= 9.3, else it checks pg_namespace).

n.b.: This defined type *requires* that a `db` is passed. This is a
concious design decision, since we find it rather useless to create such
schemas in the default `postgres` database, and if *were* useful, one
can always "over-specify".

This addresses MODULES-1098.
@igalic
Copy link
Contributor Author

igalic commented Jul 29, 2014

the travis failure is due to strict variables, i don't know if or how I'm supposed to fix that here

hunner added a commit that referenced this pull request Jul 29, 2014
defined type for creating database schemas
@hunner hunner merged commit 03a5159 into puppetlabs:master Jul 29, 2014
@igalic igalic deleted the schema branch July 29, 2014 15:08
cegeka-jenkins pushed a commit to cegeka/puppet-postgresql that referenced this pull request Oct 23, 2017
defined type for creating database schemas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants