[autotest] Simplify gather_unique_dicts logic
BUG=None
TEST=None
Change-Id: I0b658ab95b2a405db83d731f2ae37dc7ff5ba64d
Reviewed-on: https://chromium-review.googlesource.com/420513
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
diff --git a/frontend/afe/rpc_utils.py b/frontend/afe/rpc_utils.py
index 0ea1418..5b9a000 100644
--- a/frontend/afe/rpc_utils.py
+++ b/frontend/afe/rpc_utils.py
@@ -110,13 +110,10 @@
"""\
Pick out unique objects (by ID) from an iterable of object dicts.
"""
- id_set = set()
- result = []
+ objects = collections.OrderedDict()
for obj in dict_iterable:
- if obj['id'] not in id_set:
- id_set.add(obj['id'])
- result.append(obj)
- return result
+ objects.setdefault(obj['id'], obj)
+ return objects.values()
def extra_job_status_filters(not_yet_run=False, running=False, finished=False):