search

kieker-monitoring / kieker

Kieker's main repository
Apache License 2.0
70 stars 41 forks source link

[KIEKER-1023] Add a filter for session reconstruction #2822

Closed rju closed 1 week ago

rju commented 1 week ago

JIRA Issue: KIEKER-1023 Add a filter for session reconstruction Original Reporter: Holger Knoche

When considering user interaction, traces are usually too short to produce useful data. It would therefore be very helpful to have a filter which groups traces in sessions.

rju commented 1 week ago

author holgerknoche -- Fri, 2 Nov 2012 09:41:35 +0100

A first version has been created on branch SessionReconstruction, commits
1. 5ab7a235cbc71866619fd1d7e6d9862f4d549c22
1. 89112e1361ae02cd86a182ce40ad2dce00b2113a
1. b17a03c9f2388a222472ca7d6f4d15160fe4b23a
1. 00ced9848f3de5ea062cc15283ea5da31c74d855
1. b94d1f81539d70758b6dc7a9e17ee7fe1f74809d
1. 7a642d49ca21475be836ae4c51e153560f5eb815
1. dd8f383dc6da4219db0757e086b6908a28a0e9da
1. 8604c4cf5296ccd668cf18f4b2c18797276ea8fb

rju commented 1 week ago

author André van Hoorn -- Fri, 12 Apr 2013 08:55:23 +0200

See also KIEKER-952 Done

The filter needs to be integrated into HEAD

rju commented 1 week ago

author Jan Waller -- Tue, 28 May 2013 11:03:10 +0200

changed branch name to SessionReconstruction-KIEK-1023

rju commented 1 week ago

author André van Hoorn -- Mon, 29 Jul 2013 14:44:41 +0200

Merged master into branch in changeset:0d6beb6fd25ee25880ce5fe0c204296627a61922/kieker-git. The three existing tests (in TestSessionReconstructionFilter pass).

Next steps:
1. Recapture what the filter is doing exactly (roughly: collect execution/message traces with the same session ID and output after max. think time elapsed.). The JavaDoc for the class should be improved to document this.
2. Quality control/code review
3. Additional tests needed?
3. Merge back to master

rju commented 1 week ago

author nils-christian -- Mon, 28 Oct 2013 15:14:46 +0100

May be related to iObserve.

Would be nice to clean up and update to 1.9

rju commented 1 week ago

author André van Hoorn -- Mon, 30 Jun 2014 18:21:28 +0200

Replying to [avh|comment:5]:
> Merged master into branch in changeset:0d6beb6fd25ee25880ce5fe0c204296627a61922/kieker-git. The three existing tests (in TestSessionReconstructionFilter pass).
>
> Next steps:
> 1. Recapture what the filter is doing exactly (roughly: collect execution/message traces with the same session ID and output after max. think time elapsed.). The JavaDoc for the class should be improved to document this.

Data structures:

Plugins:

Tests:

> 2. Quality control/code review

> 3. Additional tests needed?

To be checked

> 3. Merge back to master

First merge master back into branch ...

rju commented 1 week ago

author André van Hoorn -- Tue, 1 Jul 2014 09:27:04 +0200

Just (re-)created a remote branch SessionReconstruction-KIEK-1023, which is based on the existing tag 1.7-SessionReconstruction-KIEK-1023.

rju commented 1 week ago

author André van Hoorn -- Tue, 1 Jul 2014 12:52:56 +0200

Replying to [avh|comment:10]:
> Replying to [avh|comment:5]:
> > Merged master into branch in changeset:0d6beb6fd25ee25880ce5fe0c204296627a61922/kieker-git. The three existing tests (in TestSessionReconstructionFilter pass).
> >
> > ...
> > 3. Merge back to master
>
> First merge master back into branch ...

Done

rju commented 1 week ago

author André van Hoorn -- Tue, 1 Jul 2014 22:21:45 +0200

Removed configuration wrapper class and added timeunit-related functionality in changeset:1cfc7dc1c7358c8a9cfe46be814caf1f0c8086df/kieker-git

Replying to [avh|comment:10]:
> Replying to [avh|comment:5]:
> > Merged master into branch in changeset:0d6beb6fd25ee25880ce5fe0c204296627a61922/kieker-git. The three existing tests (in TestSessionReconstructionFilter pass).
> >
> > Next steps:
> > 1. Recapture what the filter is doing exactly (roughly: collect execution/message traces with the same session ID and output after max. think time elapsed.). The JavaDoc for the class should be improved to document this.
>
> Data structures:
>
> * AbstractSession<T extends AbstractTrace> holds a list of traces contained in the same session. Meta-data: sessionId, startTime, endTime, and state.
> * {{ExecutionTraceBasedSession (extends }}AbstractSession<ExecutionTrace>{{).
>
> Plugins:
>
> * }}SessionReconstructionFilter{{ reads traces via its input port }}processExecutionTrace{{ (accepting events of type }}ExecutionTrace{{) and outputs sessions, which are events of type }}ExecutionTraceBasedSession{{.
>
> Tests:
> * }}TestSessionReconstructionFilter{{ (package }}kieker.test.tools.junit.traceAnalysis.filter.sessionReconstruction{{)
> * Util classes in }}kieker.test.tools.util.filter.sessionReconstruction{{
>
> > 2. Quality control/code review
>
> * }}SessionReconstructionFilterConfiguration{{: should be removed as this pattern is not used elsewhere.
> * }}AbstractSession{{: Is the implementation with }}ISessionState` over-complicating things?
> * Add generic time unit related attributes?
> * Double-check with (potential) changes to trace reconstruction filter w.r.t. timeout handling (I remember there was s.th. like this)
>
> > 3. Additional tests needed?
>
> To be checked
>
> > 3. Merge back to master
>
> First merge master back into branch ...

rju commented 1 week ago

author André van Hoorn -- Thu, 10 Jul 2014 17:27:27 +0200

Merged into master (changeset:1dbc18b3025f8c07761933d337f3baa71ea5b2a3/kieker-git)

rju commented 1 week ago

author nils-christian -- Wed, 24 Sep 2014 15:43:26 +0200

Do we still need the corresponding branch?

rju commented 1 week ago

author nils-christian -- Thu, 25 Sep 2014 12:30:40 +0200

Branch has been removed. Corresponding changeset is mentioned above.