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

document how to use super_user #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

reedstrm
Copy link
Contributor

@reedstrm reedstrm commented Sep 5, 2017

This explains the two-cursor-two-connection problem, I hope.

----------

Note the template for migrations mentions a second cursor context manager available for running steps as a
database superuser, be default `postgres`. This is useful if you want to use a less priviliged user for most
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"be default" => "by default"

By seperating into two migrations, the function created by the super_user connection is
available to the less priviliged user to create the trigger on the table owned by them.

If you wish to combine these into a single migration, it is necessary to call `commit`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was not necessary for me. See #43 (comment)

@coveralls
Copy link

coveralls commented Sep 5, 2017

Coverage Status

Coverage remained the same at 97.118% when pulling ed1515c on Connexions:readme-superuser into c8a4c96 on karenc:master.

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.

3 participants