Skip to content

Issue #1947: postgresql: set synchronous_commit to off #1970

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

Merged

Conversation

valyala
Copy link
Contributor

@valyala valyala commented Mar 15, 2016

Disable synchronous commits in postgresql as discussed at #1947 .

@valyala valyala force-pushed the postgresql-sync-commit-off branch from 30cf96a to 7b4eb1f Compare March 15, 2016 06:50
@valyala valyala force-pushed the postgresql-sync-commit-off branch 2 times, most recently from 944453f to a48ab7f Compare March 15, 2016 06:52
@valyala
Copy link
Contributor Author

valyala commented Mar 16, 2016

FYI, this PR already passed travis tests. After that I updated comments in the postgresql.conf, so new test has been queued.

BTW, it would be great if travis automatically cancels pending tests on PR update, so only the latest test is queued. Is it possible?

@valyala
Copy link
Contributor Author

valyala commented Mar 16, 2016

@msmith-techempower , could you cancel this and this test, since they are useless now and only occupy travis resources?

@valyala
Copy link
Contributor Author

valyala commented Mar 17, 2016

Ping. The PR passes tests - see https://travis-ci.org/TechEmpower/FrameworkBenchmarks/builds/116054555 .

@msmith-techempower
Copy link
Member

We have an agreement; LGTM!

msmith-techempower added a commit that referenced this pull request Mar 17, 2016
Issue #1947: postgresql: set synchronous_commit to off
@msmith-techempower msmith-techempower merged commit 1bbcd92 into TechEmpower:master Mar 17, 2016
@valyala valyala deleted the postgresql-sync-commit-off branch March 22, 2016 11:12
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.

2 participants