Sort deltas from juju core

Sort deltas from juju core

Deltas need to be processed in a certain order (service - relation - unit - machine - annotation); sort them before processing a delta.

https://code.launchpad.net/~makyo/juju-gui/sort-deltas/+merge/182697

(do not edit description out of merge proposal)

[matthew.scott, 2013-08-28 16:37:12]

Please take a look.

[rharding, 2013-08-28 17:01:28]

LGTM with test note

https://codereview.appspot.com/13350043/diff/1/test/test_env_go.js
File test/test_env_go.js (right):

https://codereview.appspot.com/13350043/diff/1/test/test_env_go.js#newcode1251
test/test_env_go.js:1251: ['service', 'deploy', {}]
I'd just like to see all 5 here to verify there's no issue in the other two
sorting into place since we've given distinct numbers to each.

[benjamin.saller, 2013-08-28 17:10:28]

LGTM, thanks

Thanks for talking this through on IRC as well

[matthew.scott, 2013-08-28 17:18:24]

Please take a look.

[benjamin.saller, 2013-08-28 17:24:03]

still LGTM. might be that if you're really defensive you'd check that a[0] and
b[0] exist before feeding that result into cmp but seems ok as is.

[matthew.scott, 2013-08-28 17:34:15]

*** Submitted:

Sort deltas from juju core

Deltas need to be processed in a certain order (service - relation - unit -
machine - annotation); sort them before processing a delta.

R=rharding, benjamin.saller
CC=
https://codereview.appspot.com/13350043