aboutsummaryrefslogtreecommitdiff
path: root/trappy/bare_trace.py
diff options
context:
space:
mode:
authorJoel Fernandes <joelaf@google.com>2017-06-29 15:16:40 -0700
committerJoel Fernandes <joelaf@google.com>2017-06-29 15:57:55 -0700
commita951463b87e0a17229d5fcd426b0b05a4ffb0764 (patch)
treefe30c6c94affc5620f752b3cf12c766484a8c5b2 /trappy/bare_trace.py
parent7f0b67931d5035205915a27cf6808707cf5b2030 (diff)
downloadtrappy-a951463b87e0a17229d5fcd426b0b05a4ffb0764.tar.gz
trappy/caching: Couple of fixes
- json params loading represents keys as unicode, use dumps to fix it - finalize_objects need to be run always for cases where some events not cached, fix it by moving it up and also handle bjackman's concern that we run it twice by skipping finalize for cached events. - fix time normalization breakage as a result of the above changes Change-Id: I2011de0ae8112e937ee61baee8d53a63d0bbe85a Signed-off-by: Joel Fernandes <joelaf@google.com>
Diffstat (limited to 'trappy/bare_trace.py')
-rw-r--r--trappy/bare_trace.py3
1 files changed, 3 insertions, 0 deletions
diff --git a/trappy/bare_trace.py b/trappy/bare_trace.py
index e4fec48..a953a60 100644
--- a/trappy/bare_trace.py
+++ b/trappy/bare_trace.py
@@ -134,6 +134,9 @@ class BareTrace(object):
def finalize_objects(self):
for trace_class in self.trace_classes:
+ # If cached, don't need to do any other DF operation
+ if trace_class.cached:
+ continue
trace_class.tracer = self
trace_class.create_dataframe()
trace_class.finalize_object()