-
Notifications
You must be signed in to change notification settings - Fork 11
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
CadcTapClient.query
should not use automatic retries
#178
Comments
More details: cadc-tap query -f tsv --debug --timeout=60 -s ivo://cadc.nrc.ca/ad "select 'cadc:CFHT', count(distinct fileName) from archive_files where archiveName = 'CFHT' and fileName not like 'CFHT_%' and fileName not like 'WIRDS_%'" Pat: The second attempt to GET /sync/{jobid}/run should not fail with a 500... that should be handled and be a 4xx (hopefully something more specific that 400 is approp), so that's a bug. |
if that's a sync query, there is a get|post to create the "job" and a redirect to a url that runs it; the http GET after the redirect runs the job (QUEUED -> EXECUTING) and if that times out you can't retry it because it already started (and you lost your connection to the output stream)... so that error is caused by retrying the GET (not supported)
Retry should start from the top by issuing another query.
Observed error on long queries:
WARNING:RetrySession:Resending request in 30s ... ERROR:: unexpected exception: java.lang.RuntimeException: ca.nrc.cadc.uws.server.JobPhaseException: cannot execute job i6yog5u4yga33kl7 when phase = EXECUTING
The text was updated successfully, but these errors were encountered: