This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 105536

Summary:	[ProfilingPoints] Fraction of time to take snapshot is included in method's execution time
Product:	profiler	Reporter:	Alexander Kouznetsov <mrkam>
Component:	Base	Assignee:	Tomas Hurka <thurka>
Status:	REOPENED ---
Severity:	blocker
Priority:	P3
Version:	6.x
Hardware:	PC
OS:	Windows XP
Issue Type:	DEFECT	Exception Reporter:
Attachments:	Snapshot without PP Snapshot with Take Snapshot PP Snapshot with Take HeapDump PP

Description Alexander Kouznetsov 2007-06-04 10:31:52 UTC

NetBeans IDE Dev (Build 200705311200)
1.6.0; Java HotSpot(TM) Client VM 1.6.0-b105
Windows XP version 5.1 running on x86
ru_RU (nb); Cp1251

Steps to reproduce:
- Run performance analysis to measure execution time of some particular method
- Insert Take Snapshot into that method
- Repeat performance analysis
- RESULT: A big amount of time that is needed for Profiler to take a snapshot is
added to method's execution time

This is another example of time inaccuracy introduced by profiling points (see
also issue 79210).

Comment 1 Tomas Hurka 2007-07-16 15:00:08 UTC

Fixed in trunk.

Comment 2 Alexander Kouznetsov 2007-08-24 13:01:24 UTC

Failed to verify with
Product Version: NetBeans IDE Dev (Build 200708211200) 
Java: 1.6.0; Java HotSpot(TM) Client VM 1.6.0-b105 
System: Windows XP version 5.1 running on x86; Cp1251; ru_RU (nb)

Method Time is increased by 63% with Take Snapshot PP, and by 155% with Take Heap Dump PP. See attached snapshots.

Comment 3 Alexander Kouznetsov 2007-08-24 13:03:15 UTC

Created attachment 47298 [details]
Snapshot without PP

Comment 4 Alexander Kouznetsov 2007-08-24 13:03:43 UTC

Created attachment 47300 [details]
Snapshot with Take Snapshot PP

Comment 5 Alexander Kouznetsov 2007-08-24 13:04:08 UTC

Created attachment 47301 [details]
Snapshot with Take HeapDump PP

Comment 6 Tomas Hurka 2007-08-27 14:49:32 UTC

Take snapshot still has some penalty, but current situation is a huge improvement against original state, when time in a range of seconds was added to the 
method execution time, currently this is around one milisecond. Lowering priority to P3.

Comment 7 Tomas Hurka 2007-09-20 15:01:34 UTC

Not for 6.0

Comment 8 Alexander Kouznetsov 2007-09-20 15:19:30 UTC

Please don't remove it070918 tags