On Fri, 03 Jul 2015 11:39:12 +0900 IWAMOTO Toshihiro <[email protected]> wrote:
> At Tue, 30 Jun 2015 22:57:32 +0900 (JST), > FUJITA Tomonori wrote: >> >> On Tue, 30 Jun 2015 17:09:48 +0900 >> IWAMOTO Toshihiro <[email protected]> wrote: >> >> > At Tue, 30 Jun 2015 17:01:54 +0900, >> > IWAMOTO Toshihiro wrote: >> >> >> >> While msgpack has default binary-unicode conversion, the feature is >> >> somehow disabled in ryu. As the other parts of ryu requires binary_type >> >> data where they directly appear on-wire, simply follow the policy. >> > >> > ryu/lib/rpc.py says: >> > >> > # note: on-wire msgpack has no notion of encoding. >> > # the msgpack-python library implicitly converts unicode to >> > # utf-8 encoded bytes by default. we don't want to rely on >> > # the behaviour though because it seems to be going to change. >> > # cf. https://gist.github.com/methane/5022403 >> > >> > , which I couldn't make sense of it because the referred document >> > mentions that unicode encoding will be the default. >> > >> > The alternative choice is to use str for these rpc methods. The >> > resulting code will be more readable, but I think that causes >> > confusion. Any ideas? >> >> I can't recall why we did the above... >> >> If I have to choose, the code using str for rpc methods looks >> nicer. But I have no strong opinion here. > > There's no question over binary data, but I'm not quite confident with > such as "callback", "write" or "send". > Please note this decision affects the Ryu API as a library. > Or, can we mark python3 support as experimental and postpone the > decision for now? I've just tried vrrp rpc with python3 and it seems to works fine. I guess that we can say we support python3 and see if someone would complain. ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
