kieker-monitoring / kieker

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

[KIEKER-986] Misleading output of Kieker's AspectJ agent #868

Open rju opened 2 weeks ago

rju commented 2 weeks ago

JIRA Issue: KIEKER-986 Misleading output of Kieker's AspectJ agent Original Reporter: Andre van Hoorn


1. Added the following aop.xml to examples/userguide/ch2--bookstore-application/src/META-INF/aop.xml (including only relevant parts):

...
                <include within="*"/>
...
               <aspect name="kieker.monitoring.probe.aspectj.operationExecution.OperationExecutionAspectFullNoGetterAndSetter"/>

1. Running from inside Eclipse, the Kieker output includes "noConfigurationFileAvailable":

May 19, 2014 11:26:54 PM kieker.monitoring.probe.aspectj.AspectJLoader noConfigurationFileAvailable
INFO: Using Kieker's AspectJLoader. This is not recommended for multi-classloader environments such as JavaEE and OSGI.
May 19, 2014 11:26:54 PM kieker.monitoring.probe.aspectj.AspectJLoader noConfigurationFileAvailable
INFO: To disable Kieker's AspectJLoader use the additional VM parameter '-Dkieker.monitoring.skipDefaultAOPConfiguration=true'.

1. However, the aop.xml configuration file is used:

avh@avh-ThinkPad-RSS:0:~/svn_work/phd_vanhoorn/talks/201405-KiekerLectureUPavia/kieker-1.9-release/kieker-1.9-bin/examples/userguide/ch2--bookstore-application $ cat /tmp/kieker-20140519-212655119-UTC-avh-ThinkPad-RSS-KIEKER-SINGLETON/kieker-20140519-212655183-UTC-000-Thread-1.dat 
$0;1400534815183128286;1.9;KIEKER-SINGLETON;avh-ThinkPad-RSS;1;false;0;NANOSECONDS;1
$1;1400534815181539061;public kieker.examples.userguide.ch2bookstore.Catalog.<init>();<no-session-id>;6387933858959982593;1400534815181289961;1400534815181292033;avh-ThinkPad-RSS;2;2
$1;1400534815192743601;public kieker.examples.userguide.ch2bookstore.CRM.<init>(kieker.examples.userguide.ch2bookstore.Catalog);<no-session-id>;6387933858959982593;1400534815192726212;1400534815192731756;avh-ThinkPad-RSS;3;2
$1;1400534815192770165;public kieker.examples.userguide.ch2bookstore.Bookstore.<init>();<no-session-id>;6387933858959982593;1400534815173069678;1400534815192760481;avh-ThinkPad-RSS;1;1
$1;1400534815193255645;public void kieker.examples.userguide.ch2bookstore.Bookstore.searchBook();<no-session-id>;6387933858959982593;1400534815193235927;1400534815193251050;avh-ThinkPad-RSS;4;1
$1;1400534815193409918;public void kieker.examples.userguide.ch2bookstore.Bookstore.searchBook();<no-session-id>;6387933858959982593;1400534815193405595;1400534815193407005;avh-ThinkPad-RSS;5;1
$1;1400534815193558451;public void kieker.examples.userguide.ch2bookstore.Bookstore.searchBook();<no-session-id>;6387933858959982593;1400534815193553955;1400534815193555608;avh-ThinkPad-RSS;6;1
$1;1400534815193701374;public void kieker.examples.userguide.ch2bookstore.Bookstore.searchBook();<no-session-id>;6387933858959982593;1400534815193696522;1400534815193698451;avh-ThinkPad-RSS;7;1
$1;1400534815193835834;public void kieker.examples.userguide.ch2bookstore.Bookstore.searchBook();<no-session-id>;6387933858959982593;1400534815193832043;1400534815193833271;avh-ThinkPad-RSS;8;1
$1;1400534815193864210;public static void kieker.examples.userguide.ch2bookstore.BookstoreStarter.main(java.lang.String[]);<no-session-id>;6387933858959982593;1400534815148065062;1400534815193844261;avh-ThinkPad-RSS;0;0
avh@avh-ThinkPad-RSS:0:~/svn_work/phd_vanhoorn/talks/201405-KiekerLectureUPavia/kieker-1.9-release/kieker-1.9-bin/examples/userguide/ch2--bookstore-application $ cat /tmp/kieker-20140519-212655119-UTC-avh-ThinkPad-RSS-KIEKER-SINGLETON/kieker.map 
$0=kieker.common.record.misc.KiekerMetadataRecord
$1=kieker.common.record.controlflow.OperationExecutionRecord

Checklist:

rju commented 1 week ago

author Jan Waller -- Tue, 20 May 2014 10:07:19 +0200

This is automatically caused by the method Nils used: noConfigurationFileAvailable()
Maybe rename this method...

rju commented 1 week ago

author nils-christian -- Tue, 20 May 2014 10:24:05 +0200

Fixed in 020631bdc2014aff99dabab15db3d21bd8466039.