tmf.ctf: Re-initialize start and endtime after transform is applied

With this the start and endtime is correct when opening the experiment
that has automatically applied timestamp transformation applied.

fixes #323

[Fixed] Incorrect start/end time in ITmfTrace instances in experiment
[Added] API in TmfTrace to reset indexer

Signed-off-by: Bernd Hufmann <[email protected]>

Author: bhufmann

ctf/org.eclipse.tracecompass.tmf.ctf.core/src/org/eclipse/tracecompass/tmf/ctf/core/trace/CtfTmfTrace.java

@@ -218,18 +218,11 @@ public void initTrace(final IResource resource, final String path, final Class<?
 
         try {
             this.fTrace = new CTFTrace(path);
-            CtfTmfContext ctx;
             /* Set the start and (current) end times for this trace */
-            ctx = (CtfTmfContext) seekEvent(0L);
-            CtfTmfEvent event = getNext(ctx);
-            if ((ctx.getLocation().equals(CtfIterator.NULL_LOCATION)) || (ctx.getCurrentEvent() == null)) {
-                /* Handle the case where the trace is empty */
-                this.setStartTime(TmfTimestamp.BIG_BANG);
-            } else {
-                final ITmfTimestamp curTime = event.getTimestamp();
-                this.setStartTime(curTime);
-                long endTime = Math.max(curTime.getValue(), fTrace.getCurrentEndTime());
-                this.setEndTime(TmfTimestamp.fromNanos(endTime));
+            CtfTmfContext ctx = resetTimes();
+
+            if (ctx == null) {
+                return;
             }
             /*
              * Register every event type. When you call getType, it will register a trace to
@@ -913,4 +906,30 @@ public Path trim(@NonNull TmfTimeRange range, @NonNull Path destinationPath, @No
         return null;
     }
 
+    @Override
+    protected synchronized void resetIndexer() {
+        super.resetIndexer();
+        resetTimes();
+    }
+
+    private @Nullable CtfTmfContext resetTimes() {
+        /* Set the start and (current) end times for this trace */
+        CtfTmfContext ctx = (CtfTmfContext) seekEvent(0L);
+        CtfTmfEvent event = getNext(ctx);
+        /* Verify context is valid. It's only invalid trace if trace.init() hasn't been called) */
+        if ((ctx != null) && (ctx.getLocation() != null)) {
+            if ((ctx.getLocation().equals(CtfIterator.NULL_LOCATION)) || (ctx.getCurrentEvent() == null)) {
+                /* Handle the case where the trace is empty */
+                this.setStartTime(TmfTimestamp.BIG_BANG);
+            } else {
+                final ITmfTimestamp curTime = event.getTimestamp();
+                this.setStartTime(curTime);
+                long endTime = createTimestamp(fTrace.getCurrentEndTime()).getValue();
+                endTime = Math.max(curTime.getValue(), endTime);
+                this.setEndTime(TmfTimestamp.fromNanos(endTime));
+            }
+            return ctx;
+        }
+        return null;
+    }
 }

tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/trace/TmfTrace.java

@@ -222,6 +222,22 @@ protected ITmfTraceIndexer createIndexer(int interval) {
         return new TmfCheckpointIndexer(this, interval);
     }
 
+    /**
+     * Reset indexer. It dispose the old indexer and creates a new one,
+     *
+     * Note:
+     * Only call this when there is no trace reading is ongoing. Caller
+     * needs to ensure that trace indexing is restarted.
+     *
+     * @since 10.2
+     */
+    protected synchronized void resetIndexer() {
+        if (fIndexer != null) {
+            fIndexer.dispose();
+        }
+        fIndexer = createIndexer(fCacheSize);
+    }
+
     // ------------------------------------------------------------------------
     // ITmfTrace - Initializers
     // ------------------------------------------------------------------------
@@ -379,6 +395,7 @@ public synchronized void dispose() {
         ByteBufferTracker.setMarked();
     }
 
+
     // ------------------------------------------------------------------------
     // ITmfTrace - Basic getters
     // ------------------------------------------------------------------------

tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/trace/experiment/TmfExperiment.java

@@ -366,6 +366,7 @@ public void initExperiment(final Class<? extends ITmfEvent> type,
 
                 if (!newTransform.equals(currentTransform)) {
                     TmfTraceManager.deleteSupplementaryFiles(trace);
+                    resetIndexer();
                     trace.setTimestampTransform(newTransform);
                 }
             });