Skip to content

Commit

Permalink
Merge pull request #145 from mfine/mfine-inject-tags-into-logs
Browse files Browse the repository at this point in the history
Allow the logger to inject tags into the logs
  • Loading branch information
mookerji committed May 7, 2015
2 parents 9960e84 + b074b58 commit 966e09c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion python/sbp/client/loggers/base_logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,11 @@ class BaseLogger(object):
filename : string
File to log to.
"""
def __init__(self, filename, mode="w", dispatcher=dispatch):
def __init__(self, filename, mode="w", tags=None, dispatcher=dispatch):
self.handle = open(filename, mode)
self.dispatcher = dispatcher
self.base_time = time.time()
self.tags = dict([tuple(t.split('=')) for t in tags.split(',')]) if tags else {}

def __enter__(self):
return self
Expand Down
3 changes: 2 additions & 1 deletion python/sbp/client/loggers/json_logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,8 @@ def fmt_msg(self, msg):
data = msg.to_json_dict()
return {"delta": self.delta(),
"timestamp": self.timestamp(),
"data": data}
"data": data,
"metadata": self.tags}

def call(self, msg):
try:
Expand Down

0 comments on commit 966e09c

Please sign in to comment.