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

1073 here pull token expired error #1074

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

gabrielwol
Copy link
Collaborator

@gabrielwol gabrielwol commented Sep 17, 2024

What this pull request accomplishes:

  • add retries for access_token while maintaining same request_id
  • remove some duplicate function definitions (with some extra work pull_here and pull_here_path could be further deduped)
  • Apply similar task.bash structure to pull_here_path that already existed in pull_here.

Issue(s) this solves:

What, in particular, needs to reviewed:

What needs to be done by a sysadmin after this PR is merged

  • Update data_scripts links

@gabrielwol gabrielwol self-assigned this Sep 17, 2024
@gabrielwol gabrielwol linked an issue Sep 17, 2024 that may be closed by this pull request
@gabrielwol
Copy link
Collaborator Author

As I watch a request run for 2+ hours, I was wondering how many retries we need... Here's an analysis of the 6 failures from the last two weeks and how long the requests ultimately took to receive (via emails)!

RequestID Request Sent Request Received Delta
584506 17:01:05 19:21:56 2:20:51
584894 10:31:07 17:15:01 6:43:54
584946 17:01:06 18:00:04 0:58:58
585386 9:52:19 14:44:45 4:52:26
583954 17:01:06 18:31:20 1:30:14

-> going to set the limit to 8 hours.

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.

HERE pull token_expired error Refresh authentication token when waiting for queries longer than 60 minutes
1 participant