Skip to content
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

cpickle vs simplejson #30

Closed
ddurieux opened this issue May 25, 2015 · 4 comments
Closed

cpickle vs simplejson #30

ddurieux opened this issue May 25, 2015 · 4 comments
Assignees
Milestone

Comments

@ddurieux
Copy link
Contributor

I have seen many performances pages about thi son the web and we can see all result (cpickle > simplesjon and other cpickle < simplejson).

Somebody has made performances tests with both in alignak (currently use cpickle)?

If no, I can try it

@Seb-Solon
Copy link
Contributor

well i you want to compare, try to compare the c version of json (usjon I thnk) I will be fair vs cPickle ;)

@ddurieux
Copy link
Contributor Author

I have tested ujson and simplejson and we have problem because have objects in dict to serialize.

For example, I have in a dict

'contacts': <shinken.objects.contact.Contacts object at 0x808eaa6d0

So why not put in dict only data and the module use the object to assicate data with object. The good point for this is the data to serialize and size will be more small to pass between module via http/https.

@ddurieux ddurieux self-assigned this May 25, 2015
@Seb-Solon Seb-Solon mentioned this issue Jun 16, 2015
14 tasks
@Seb-Solon
Copy link
Contributor

We could start with define dumps and loads function for every item (in Item?) so that we can call json on it

Seb-Solon pushed a commit that referenced this issue Sep 4, 2015
Enh: Replace cpickle in brok by ujson because better performances #30
@Seb-Solon Seb-Solon mentioned this issue Nov 28, 2015
11 tasks
@ddurieux
Copy link
Contributor Author

ddurieux commented May 9, 2016

cpickle removed to json in #294

@ddurieux ddurieux closed this as completed May 9, 2016
@ddurieux ddurieux added this to the 1.0 milestone Sep 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants