Hi Andrew

THank you for making this. This looks like a great start.

I made a small PR regarding the cache.close method.

Since signals are synchronous (for now?) I think you do need the wrapper.

What about implementations of the sync methods? You could make them raise
an exception (async only) or use async_to_sync.

Good luck with your exams,

Adam

On Tue, 6 Oct 2020 at 19:25, Andrew Wang <acwangpyt...@gmail.com> wrote:

> Hi all,
>
> 1.5 weeks ago, I posted that I was interested in making Django's cache
> async. Just here to report that the package for django-async-redis is on
> pypi and the repo is here:
> https://github.com/Andrew-Chen-Wang/django-async-redis. Please test it
> out. I have only tested it on a local Django project with uvicorn as a test
> server.
>
> Regarding edge cases from our previous conversation, there were none that
> I could see. I've stuck with the naming scheme get_async rather than
> async_get. I think I'll leave it like that for the foreseeable future (I
> didn't feel a difference in typing speed, either).
>
> My only concern is cache.close() which can be fixed in a PR for adding
> cache.close_async in BaseCache:
> https://github.com/django/django/blob/999cddd58d30469f3ee85278985313fdf528323d/django/core/cache/__init__.py#L116-L121
>
> In other words, I don't think Django's caching framework needs to
> change... for most of it besides those aforementioned lines. That's all
> I've got for now (midterms are tomorrow). Let me know if there are any
> critical failures.
>
> Cheers,
> Andrew
>
> --
> You received this message because you are subscribed to the Google Groups
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to django-developers+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/django-developers/9eabc712-a33a-4d93-a326-3a683d11c62bn%40googlegroups.com
> <https://groups.google.com/d/msgid/django-developers/9eabc712-a33a-4d93-a326-3a683d11c62bn%40googlegroups.com?utm_medium=email&utm_source=footer>
> .
>


-- 
Adam

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/CAMyDDM3gUkUWO3v3HbkGEeZO-i2fXTQaRQxyhv2cFJXZ4490kg%40mail.gmail.com.

Reply via email to