-
Notifications
You must be signed in to change notification settings - Fork 20
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
mysterious E275 error message repeatedly showing up in KG2 logfiles of services under heavy load #2141
Comments
Perhaps I'm wrong and this is truly an unexpected (or not easily explainable) condition, in which case, we should dig deeper. :-) |
I'll check @saramsey |
Yes, the E275 is my fault. I did not understand why this condition was happening and thus put in this lame message. RTX/code/ARAX/ARAXQuery/ARAX_query.py Lines 329 to 333 in 8c435f0
and I don't think it should be hitting the E275 code during over limits, but I'm not certain. If would be good if we can understand what is happening and handle it more gracefully. I did not get to it. |
Thank you @edeutsch for giving the historical perspective here. |
Can we close out this issue now, @edeutsch ? |
I was planning on labeling this [technical debt] and investigating a bit better and fixing it. I think it is not resolved. |
We frequently see error messages like this in the
/tmp/RTX_OpenAPI_kg2.elog
and/tmp/RTX_OpenAPI_kg2beta.elog
files:I think it is coming from this line of code:
RTX/code/ARAX/ARAXQuery/ARAX_query.py
Line 268 in 8c435f0
I think this condition is actually expected to occur whenever we respond to a client request with a HTTP status code of
429
, since in that case we are not processing the query and thus there is nojob_id
. I think. If that is true, then could we please update the error message so it doesn't look like something anomalous that we need to debug? Maybe change to an INFO message or something.The text was updated successfully, but these errors were encountered: