-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extract adisp and upload it to pypi #11
Comments
As a reverse opinion, maybe swirl should be taken into consideration, too? It does (almost) the same thing, just with a bit different syntax, and I got the impression that it's the most preferred async sugar in Tornado community. Swirl lacks |
Yes, switching to swirl is an another option. Adisp though is also well-known (at least in Russian python community) and proven library (it appears a couple of week earlier than swirl) with nice API and implementation so this is not a variation of NIH sindrome ;) Adisp is available at launchpad : https://launchpad.net/adisp but the version there does not include evilkost's tornado fixes and it is not uploaded to pypi. All these reasons are community-related. Adisp community is just not as visible as swirl's (because of launchpad vs github, Russian blog post vs English blog post) but this community exists for sure. I don't think swirl should be considered 'standard solution' while adisp being an incarnation of NIH sindrome, they are both used by many people. Besides the community there are technical reasons. The lack of 'async' decorator is not a serious issue because it can be implemented in a couple of lines. I don't have an experience with swirl and my experience with adisp is quite limited (while being positive) so technical advantages are not clear for me and this need some investigation. Both libraries work. So I don't have neither community nor technical arguments for adisp vs swirl question :) It is easier to extract adisp than to convert brukva to swirl and so without better arguments it seems preferrable to extract adisp. |
What do you think about extracting tornado-specific adisp library from brukva and releasing it under e.g. 'adisp-tornado' name?
The text was updated successfully, but these errors were encountered: