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

Fix #27655: handling of success in postgres_local_cache #27656

Merged
merged 1 commit into from Oct 5, 2015

Conversation

Projects
None yet
3 participants
@gracinet
Contributor

gracinet commented Oct 3, 2015

  • using get() is anyway more pythonic
  • psycopg2 translates None to NULL
  • moving explanation to schema comment

(there's no existing unit test for this)

Fix #27655: handling of success in postgres_local_cache
- using get() is anyway more pythonic
- psycopg2 translates None to NULL
- moving explanation to schema comment

(there's no existing unit test for this)
@jfindlay

This comment has been minimized.

Show comment
Hide comment
@jfindlay

jfindlay Oct 5, 2015

Contributor

Fix #27655.

Contributor

jfindlay commented Oct 5, 2015

Fix #27655.

@rallytime

This comment has been minimized.

Show comment
Hide comment
@rallytime

rallytime Oct 5, 2015

Contributor

Thanks @gracinet!

Contributor

rallytime commented Oct 5, 2015

Thanks @gracinet!

rallytime added a commit that referenced this pull request Oct 5, 2015

Merge pull request #27656 from gracinet/27655-pg-local-cache-success
Fix #27655: handling of success in postgres_local_cache

@rallytime rallytime merged commit f42482e into saltstack:develop Oct 5, 2015

5 checks passed

default Merged build finished.
Details
jenkins/salt-pr-clone Salt PR - Clone Repository #10002 — SUCCESS
Details
jenkins/salt-pr-linode-ubuntu14.04-n Salt PR - Linode Ubuntu 14.04 #1075 — SUCCESS
Details
jenkins/salt-pr-lint-n Salt PR - Code Lint #9720 — SUCCESS
Details
jenkins/salt-pr-rs-cent7-n Salt PR - RS CentOS 7 #8549 — SUCCESS
Details

basepi added a commit that referenced this pull request Oct 5, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment