#36879: Add Django cache identification to Redis client metadata
-------------------------------------+-------------------------------------
Reporter: Vasil Chomakov | Owner: (none)
Type: New feature | Status: new
Component: Core (Cache system) | Version: dev
Severity: Normal | Resolution:
Keywords: redis cache | Triage Stage:
observability | Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Zachary W (ar3ph)):
Replying to [comment:6 Vasil Chomakov]:
> I suggest using `DriverInfo` when available (can be successfully
imported) and falling back to `lib_name` and `lib_version` if `redis-py`
does not yet support it.
>
> IMPORTANT: Note that when manually set, `lib_name` and `lib_version`
should look like this:
> * `lib_name` = `redis-py(django-cache_v{django-cache-version})`
> * `lib_version` = `{redis-py-version}`
> per Redis `CLIENT SETINFO` documentation:
https://redis.io/docs/latest/commands/client-setinfo/
Should the driver be called `django-cache`? My opinion is to just call it
`django`, b/c `django-cache` is not a separated package.
--
Ticket URL: <https://code.djangoproject.com/ticket/36879#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion visit
https://groups.google.com/d/msgid/django-updates/0107019bf07bcc48-a4b11e38-bb51-4470-8721-7bf4d411066a-000000%40eu-central-1.amazonses.com.
