Skip to content

bug fix: formatting exception so it can be saved to the database. #1480

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

Closed
wants to merge 4 commits into from

Conversation

ruslanstarikov
Copy link

I have encountered the problem with saving failed jobs to mongo, - the exception field is always blank.
The problem was reproduced using amqp and mysql as queue drivers, and mongo db version 3.6.2 on ubuntu 14.04.
I have formatted the exception field into the array with the message and stack trace as keys.
This way exception can be saved to the mongo db database.

@divine
Copy link
Contributor

divine commented Jan 20, 2020

Hello!

Could you please take care for conflicting file or we can take a look?

Thank you for your contribution!

@ruslanstarikov
Copy link
Author

Done. Please review.

@Smolevich
Copy link
Contributor

@ruslanstarikov
Can you explain profit of this changes?
If you add tests, it will be very good

@ruslanstarikov
Copy link
Author

I'll add some tests this weekend. I did it almost two years ago.
The purpose of this patch is to make sure that the exception is persisted into the failed_jobs record, same way it does into the SQL databases. Without the patch, the exception field will always be blank.

@divine
Copy link
Contributor

divine commented Jan 31, 2020

@ruslanstarikov any update?

Thank you!

@divine
Copy link
Contributor

divine commented Feb 16, 2020

Closing in favor of #1961

@divine divine closed this Feb 16, 2020
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.

3 participants