[ovs-dev] [PATCH] JSON serialization via Python's json lib

Ben Pfaff blp at ovn.org
Tue Jul 26 16:19:13 UTC 2016


On Mon, Jul 25, 2016 at 07:17:11PM -0500, Terry Wilson wrote:
> There is no particularly good reason to use our own Python JSON
> serialization implementation when serialization can be done faster
> with Python's built-in JSON library.
> 
> A few tests were changed due to Python's default JSON library
> returning slightly more precise floating point numbers.
> 
> Signed-off-by: Terry Wilson <twilson at redhat.com>

Applied, thanks.

This patch raises the question of whether OVS should be in the business
of testing Python's built-in libraries.  That is, maybe the Python
versions of the JSON tests should go away entirely?  I don't know.

Along the same lines, maybe the code that uses the OVS json python
module, at least for serialization, should be changed to just use the
Python built-in json module directly.

None of this is urgent or important.



More information about the dev mailing list