[Bug 1737989] Re: [MIR] ujson?

No longer required - ujson has been dropped from ceilometer and aodh ** Changed in: ujson (Ubuntu) Status: Incomplete => Invalid ** Changed in: aodh Status: New => Invalid -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to

[Bug 1737989] Re: [MIR] ujson?

So, what is the outcome of this? ** Changed in: ujson (Ubuntu) Assignee: (unassigned) => James Page (james-page) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1737989 Title: [MIR] ujson? To

[Bug 1737989] Re: [MIR] ujson?

Reviewed: https://review.openstack.org/530891 Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=05050e4b2110d0807257c2a29c4ab6080813f36d Submitter: Zuul Branch:master commit 05050e4b2110d0807257c2a29c4ab6080813f36d Author: James Page Date:

[Bug 1737989] Re: [MIR] ujson?

Fix proposed to branch: master Review: https://review.openstack.org/530891 ** Changed in: ceilometer Status: New => In Progress ** Changed in: ceilometer Assignee: (unassigned) => James Page (james-page) -- You received this bug notification because you are a member of Ubuntu Bugs,

[Bug 1737989] Re: [MIR] ujson?

I'll propose a review for ceilometer; with the current state of ujson development we need to make an openstack level decision as to whether ujson should be used or not - if the performance gains are worth it, and we can sort out the maintenance concerns then maybe switch all of openstack from

[Bug 1737989] Re: [MIR] ujson?

i proposed revert from aodh just to simplify the scope: https://review.openstack.org/#/c/528829 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1737989 Title: [MIR] ujson? To manage notifications

[Bug 1737989] Re: [MIR] ujson?

when we originally switched to it in gnocchi[1], a former colleague and i were benchmarking its performance but i seemed to have removed my benchmarks when i updated my computer. i would say http://artem.krylysov.com/blog/2015/09/29/benchmark-python-json- libraries/ is an accurate representation.

[Bug 1737989] Re: [MIR] ujson

Reading the original inclusion commit: https://github.com/openstack/requirements/commit/f610e12d67fc093b713c442da2d973da6d71e0ab being actively maintained is one of the review criteria; I've raised aodh and ceilometer tasks as it would appear that this might not be the case today (things where

[Bug 1737989] Re: [MIR] ujson

is this package really maintained upstream? looking at https://github.com/esnme/ultrajson at the open issues and pull requests for segfaults and memory leaks, this isn't much promising. Granted, it's an upstream patch in ceilometer, but is this usage really performance critical? -- You received

[Bug 1737989] Re: [MIR] ujson

Bug subscription added for ubuntu-openstack -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1737989 Title: [MIR] ujson To manage notifications about this bug go to: