-
-
Notifications
You must be signed in to change notification settings - Fork 70
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
unresponsive (from error?) #237
Conversation
I'm not sure about the pickling error, but for the first error what version of MongoDB is aleph.sagemath running? A brief google search on the traceback message suggests that pre-1.8 versions have this issue. |
It's running mongodb-linux-x86_64-1.8.1 |
Hmm, then it seems like someone's input just generated a massive output message, and the DB complained about it being larger than 16MB (hence the error). Implementing gh-139 could be a good long-term solution, but the quickest solution seems to be to just catch the error when things are added to the database and drop the message. Or, do we want to find a way to let the user know that their output was too large / was rejected? |
This just halted all of the workers again. As a quick fix, let's just drop the computation (or if there's an easy way, insert a message back to the user saying there was an error). |
Nice, and I agree with Alex's comment. And I'm really curious: how are you attaching code to an existing issue? You can do that from the API; are you using the API? |
Yes, I’ve been using the Github API. |
Do you happen to have a shell script or python script you could share with us? I know it's easy enough to write my own, but I thought maybe you might have something already written. |
"sequence": m["sequence"]}) | ||
log("INSERTED: %s"%('\n'.join(str(m) for m in success),)) | ||
if len(success) < len(messages): | ||
log("FAILED TO INSERT %d message(s)" % (len(messages) - len(success))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, there is a tab on this line. That needs to be changed to spaces.
… referee fixes by jasongrout) The problem was that messages were exceeding the mongodb limits for record size. Now we just insert an error in the message stream when that happens
I guess that's what I get for adding a few small referee commits and merging---now that it didn't contain your tab commit, it appears that I didn't merge your branch. But I made the same tab fix and a few other fixes that relate to printing enormous messages in the logs. |
aleph.sagemath.org is now unresponsive, and here is the last bit of the stdout log: