Skip to content

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

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

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

Conversation

@sp1rs

@sp1rs sp1rs commented Jul 11, 2019

Copy link
Copy Markdown
Contributor

Fixes #5301

@auvipy auvipy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for tackling this

Comment thread docs/django/first-steps-with-django.rst Outdated
# django setting.
CACHES = {
'default': {
'BACKEND': 'api.lib.redis_cache_backend.RedisCache',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

api??

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@codecov

codecov Bot commented Jul 11, 2019

Copy link
Copy Markdown

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