What about backends such as MongoDB which override _store_result?

What about backends such as MongoDB which override _store_result?

I have changed the mongodb one also!

It shouldn't be hard to add the same capability to the Cassandra backend.
You'll need to edit these queries.
Add a condition here, here and here to use the correct table creation, insert statement and read statement according to the configuration value.
CQL is very much like SQL in this regard.

Thanks @geoffreybauduin. I am trying to run the test locally to fix the tests actually!

hi guys,
do celery Result Stores support MongoDB. Thanks you vvery much

@sckhg1367 MongoDB is supported, probably the documentation needs to be updated:

https://github.com/celery/celery/blob/master/celery/backends/mongodb.py

Would you like to open a PR perhaps to include all available result stores?

Thanks @geoffreybauduin.

Any updates on this? This is a crucial fix and looks like we are pretty close here. Can someone please prioritize this?

I'll work on it.

Sorry I could not get time to work on this! @thedrow Can you let me know what do you think about failing tests?

I'll rebase first and then let you know.

Great thanks guys for taking a look into this.

I'm not sure I resolved the conflict correctly which is why I merged.
Can anyone take a look?

seems only 1 failure

@thedrow I think rebase is appropiate here. I will rebase it upon master and will take a look at the test failure.

Any updates here? Can someone please share the estimated timeline for releasing Milestone 4.4.0 ? Please let me know if I can help in some way, thanks.

30th Nov

@auvipy Which tests you are referring?

https://github.com/celery/celery/tree/master/t/unit/backends test_base, test_database, test_mongo

@safwanrahman Are you working on the tests? I haven't contributed to this project before but I could try to help if you need any. Please let me know if we can still make it to the Milestone 4.4.0 release.

@safwanrahman Are you working on the tests? I haven't contributed to this project before but I could try to help if you need any. Please let me know if we can still make it to the Milestone 4.4.0 release.

you can push some tests on his forks branch :)

@auvipy @safwanrahman While I am touching this branch, just thought of checking with you guys if it would make sense to create a __init__ method/constructor in this class as we have changes in this PR here and we can leverage this parent init via super here.

@safwanrahman Are you working on the tests? I haven't contributed to this project before but I could try to help if you need any. Please let me know if we can still make it to the Milestone 4.4.0 release.

you can push some tests on his forks branch :)

@safwanrahman @auvipy I few tests that I would like to push but I don't have permissions to push to your fork. Please let me know how you would like me this few changes.

I will be out for the week so just created the PR to your branch here, please feel free to update if you would like to proceed meanwhile. Thanks.

@akshjain83 Thanks for the commit, sorry I did not get time to write the tests myself. I have cherry-picked the commit and pushed to the branch.
@auvipy possible to merge?

No worries @safwanrahman Happy to contribute. @auvipy How does the timeline looks like for milestone 4.4.0 ? Any updated estimates?

I'm hoping to make a release by the weekend.

Thanks @thedrow