[openstack-dev] [oslo] oslo.serialization repo review
Ben Nemec
openstack at nemebean.com
Wed Jul 16 20:15:44 UTC 2014
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 07/15/2014 03:52 PM, Ihar Hrachyshka wrote:
> On 15/07/14 20:36, Joshua Harlow wrote:
>> LGTM.
>
>> I'd be interesting in the future to see if we can transparently
>> use some other serialization format (besides json)...
>
>> That's my only compliant is that jsonutils is still named
>> jsonutils instead of 'serializer' or something else but I
>> understand the reasoning why...
>
> Now that jsonutils module contains all basic 'json' functions
> (dump[s], load[s]), can we rename it to 'json' to mimic the
> standard 'json' library? I think jsonutils is now easy to use as an
> enhanced drop-in replacement for standard 'json' module, and I even
> envisioned a hacking rule that would suggest to use jsonutils
> instead of json. So appropriate naming would be helpful to push
> that use case.
We discussed this a bit on the oslo.utils spec, but we don't want to
shadow builtin names so we're leaving the utils suffix on the modules
that have it. I would think the same applies here.
If someone wants to use this as a dropin they can still do "from
oslo.serialization import jsonutils as json"
>
> /Ihar
>
>
>> -Josh
>
>> On Jul 15, 2014, at 10:42 AM, Ben Nemec <openstack at nemebean.com>
>> wrote:
>
>>> And the link, since I forgot it before:
>>> https://github.com/cybertron/oslo.serialization
>>>
>>> On 07/14/2014 04:59 PM, Ben Nemec wrote:
>>>> Hi oslophiles,
>>>>
>>>> I've (finally) started the graduation of oslo.serialization,
>>>> and I'm up to the point of having a repo on github that
>>>> passes the unit tests.
>>>>
>>>> I realize there is some more work to be done (e.g. replacing
>>>> all of the openstack.common files with libs) but my plan is
>>>> to do that once it's under Gerrit control so we can review
>>>> the changes properly.
>>>>
>>>> Please take a look and leave feedback as appropriate.
>>>> Thanks!
>>>>
>>>> -Ben
>>>>
>>>
>>>
>>> _______________________________________________ OpenStack-dev
>>> mailing list OpenStack-dev at lists.openstack.org
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>>>
>>>
>
>> _______________________________________________ OpenStack-dev
>> mailing list OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>>
>
> _______________________________________________ OpenStack-dev
> mailing list OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQEcBAEBAgAGBQJTxt1rAAoJEDehGd0Fy7uqm0EH/RwwkuCOwrZy/f/DhxgHgyXA
zWi9x29M+q9kDdkJImdoSCoimReV1tXGMBe/hMtqiqa7XUtC0daltPDsDgZX1rCE
Od1luXfnD8jxdIWI+6ecDpf8eK3PZqe++FHditOEVDNN6R84xW6Zkkd/3ERipT5D
Jt4G1VBV6DmeO80p94InunAvlG6f15t1NuWfqo7a1fU8r9XpKRnYqmgSBrjNxZcL
8cDTW/3HH6X2kps1xVDJTDFCo2WionbK73N9FYy1NBRt0XKThseRVXQiC4sANlEN
/tHqlWVGZg6e6HCkvywV4gAUKnaNiuHVi6U0RDgz4KIa2Qrbazup3Azz2fsbt6U=
=39+D
-----END PGP SIGNATURE-----
More information about the OpenStack-dev
mailing list