Skip to content

Fix 5301: Enchance doc for django celery backend.#5639

Merged
auvipy merged 3 commits intocelery:masterfrom
sp1rs:master
Jul 11, 2019
Merged

Fix 5301: Enchance doc for django celery backend.#5639
auvipy merged 3 commits intocelery:masterfrom
sp1rs:master

Conversation

@sp1rs
Copy link
Contributor

@sp1rs sp1rs commented Jul 11, 2019

Fixes #5301

Copy link
Member

@auvipy auvipy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for tackling this

# django setting.
CACHES = {
'default': {
'BACKEND': 'api.lib.redis_cache_backend.RedisCache',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

api??

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a random path.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/path/to/cachebackend/DBbackend

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CACHES = {
'default': {
'BACKEND': 'django.core.cache.backends.db.DatabaseCache',
'LOCATION': 'my_cache_table',
}
}

got from https://docs.djangoproject.com/en/2.2/topics/cache/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@codecov
Copy link

codecov bot commented Jul 11, 2019

Codecov Report

Merging #5639 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5639   +/-   ##
=======================================
  Coverage   83.43%   83.43%           
=======================================
  Files         144      144           
  Lines       16665    16665           
  Branches     2070     2070           
=======================================
  Hits        13904    13904           
  Misses       2547     2547           
  Partials      214      214

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f81972d...1fb8296. Read the comment docs.

@auvipy auvipy merged commit 8e016e6 into celery:master Jul 11, 2019
@auvipy auvipy added this to the 4.4.0 milestone Jul 11, 2019
jeyrce pushed a commit to jeyrce/celery that referenced this pull request Aug 25, 2021
* Fix 5301: Enchance doc for django celery backend.

* Added generic path for cache backend.

* Change path to django cache and added myself as contributor.
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.

docs: undocumented use of CELERY_CACHE_BACKEND in django-celery-results

3 participants