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

Wait a bit more after fetching a non-existant yet diff. #343

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

Conversation

Frankkkkk
Copy link

When applying diffs real time (database already up to date), the
minutely diff will only be available after a minute. So wait at least 40
secs instead of 10; less useless load on the replication server.

When applying diffs real time (database already up to date), the
minutely diff will only be available after a minute. So wait at least 40
secs instead of 10; less useless load on the replication server.
@mmd-osm
Copy link
Contributor

mmd-osm commented Nov 28, 2016

I'd suggest to have a configurable parameter like in #245 with default of 10 seconds as today.

@Frankkkkk
Copy link
Author

That seems good; however I would suggest to increase the default sleep value as:

  • In the worst case scenario, the offset is of one diff
  • Servers are "quick enough" to process the diff in 40s: 1min - apply_time < 40s

Il copy that into the other issue ;-)

@mmd-osm
Copy link
Contributor

mmd-osm commented May 25, 2017

A better solution is now available in #403

I'd suggest to close this one in favor of a parametrized solution, offering more flexibility.

@Frankkkkk
Copy link
Author

In that case I would propose to change 10seconds to 40 in #403

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.

2 participants