-
Notifications
You must be signed in to change notification settings - Fork 163
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
the future of happybase #233
Comments
i've transferred my |
I should have my third idea for merging the two forks (where async is the default) fleshed out by tomorrow. With that finished, it wouldn't be very hard at all to create a happybase package that just inverts the imports of the aiohappybase package to provide the strict supperset without code duplication that could serve as a nice transition period. |
I've finished the "sync as a sub-package" implementation: https://github.com/aiudirog/aiohappybase/pull/3 It's a little hacky in its pursuit of code dryness but I think that's okay because in theory it will nearly guarantee the two implementations don't diverge without having to code everything twice. |
this is fantastic, thank you for all your work |
considering that:
aiohappybase
work modernizes the code to be python3.6+ onlyit may be good for happybase's future if:
aiohappybase
fork if it becomes a strict superset of the original happybase, which seems technically feasible (https://github.com/aiudirog/aiohappybase/pull/1, https://github.com/aiudirog/aiohappybase/pull/2)thoughts welcome, especially from @aiudirog (with whom i also have had recent email contact).
The text was updated successfully, but these errors were encountered: