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:55:07 -0700
commitcd7f2e045fa07bb14433a6506ab8756e94cd0f19 (patch)
tree3469bbd029f3333540ab8837fa958d3dbe904bb7 /trappy/bare_trace.py
parent180ba383f2ab4e544273a1e75e8bee61df654ceb (diff)
downloadtrappy-cd7f2e045fa07bb14433a6506ab8756e94cd0f19.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()