Skip to content
This repository has been archived by the owner on Oct 31, 2018. It is now read-only.

Performance #42

Closed
jooadam opened this issue Aug 7, 2016 · 2 comments
Closed

Performance #42

jooadam opened this issue Aug 7, 2016 · 2 comments

Comments

@jooadam
Copy link

jooadam commented Aug 7, 2016

jooadam@shulur:~$ time curl "https://api.linode.com?api_key=$key&api_action=avail.datacenters"

...

real    0m0.767s
user    0m0.016s
sys     0m0.008s

jooadam@shulur:~$ time linode locations

...

real    0m3.615s
user    0m0.996s
sys     0m0.088s

Although support tries to convince me, I refuse to believe that parsing half a kilobyte of JSON and printing a single line per array item is justified to take nearly three seconds, hence I treat this as a bug.

Please let me know if additional information is needed.

@ddevault
Copy link

ddevault commented Aug 7, 2016

That seems like a fair assessment, but tbh I don't think we'll be doing more than maintenance work on this repository right now. In the course of our work on API 4 we've put together a new CLI.

Of course, patches welcome if you'd like to take a whack at improving this version.

@jooadam
Copy link
Author

jooadam commented Aug 7, 2016

@SirCmpwn Okay, fair enough. I will keep using the Manager then, until v4 arrives.

@jooadam jooadam closed this as completed Aug 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants