Increase scan reliability #3262
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Handle exceptions better - Prevent search_workers from even fully exiting - High-load backoff is now backoff based instead of randomly assigned absolute - Logins will retry `login-retries` times before throwing an exception to cause the search_worker to create a new PgoAPI instance and start over
|
What's the purpose of the |
|
Essentially, I've had instances where the PgoAPI * I "turn it off and on again" essentially |
|
You want to say that I don't see message about 15minutes ban anymore? ^_^ |
invisiblek
added a commit
that referenced
this pull request
Aug 3, 2016
* Remove unused scannedLabel() function * Remove pogom/pgoapi if empty at startup * use pgoapi from requirements.txt instead of copying its files * gitignore src/ * Fixups for using mainline pgoapi * Update pgoapi to latest HEAD * Use get_pos_by_name from pgoapi * Update search api for new pgoapi * Removing cell ID's, this functionality is provided in pgoapi * Update search for the new pgoapi (again) * making sure we change with step location, rather than starting position * silenced logging * Increase scan reliability (#3262) - Handle exceptions better - Prevent search_workers from even fully exiting - High-load backoff is now backoff based instead of randomly assigned absolute - Logins will retry `login-retries` times before throwing an exception to cause the search_worker to create a new PgoAPI instance and start over * Update pgoapi * Show a big scary warning if user hasn't updated * Use standard logging for this and for the check of the old pogom/pgoapi * Automatically remove pogom/pgoapi, regardless if its empty or not. Its not going to be useful anymore and will cause problems.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
login-retriestimes before throwing an exception to the search_worker thus creating a new PgoAPI instance and starting over