Rfaulk has submitted this change and it was merged.

Change subject: add. get functionality.
......................................................................


add. get functionality.

Change-Id: Ibb05fc4f3b9adb66a06440695551cc61e1d0b788
---
M user_metrics/api/broker.py
1 file changed, 11 insertions(+), 2 deletions(-)

Approvals:
  Rfaulk: Verified; Looks good to me, approved



diff --git a/user_metrics/api/broker.py b/user_metrics/api/broker.py
index 0b4d2e5..65d16c8 100644
--- a/user_metrics/api/broker.py
+++ b/user_metrics/api/broker.py
@@ -103,6 +103,15 @@
             for line in lines:
                 f.write(line)
 
-
     def get(self, target, key):
-        pass
+        """
+        Retrieve a value with the given key
+        """
+        with open(target, 'r') as f:
+            lines = f.read().split('\n')
+            for idx, line in enumerate(lines):
+                item = json.loads(line)
+                if item.keys()[0] == key:
+                    return item[key]
+        return None
+

-- 
To view, visit https://gerrit.wikimedia.org/r/72696
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb05fc4f3b9adb66a06440695551cc61e1d0b788
Gerrit-PatchSet: 1
Gerrit-Project: analytics/user-metrics
Gerrit-Branch: repair_runtime
Gerrit-Owner: Rfaulk <[email protected]>
Gerrit-Reviewer: Rfaulk <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to