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 139399 - Extend the Log with time measuring handler
Summary: Extend the Log with time measuring handler
Status: RESOLVED WONTFIX
Alias: None
Product: platform
Classification: Unclassified
Component: NB JUnit (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Jaroslav Tulach
URL:
Keywords: API_REVIEW_FAST
Depends on:
Blocks:
 
Reported: 2008-07-09 14:34 UTC by Pavel Flaska
Modified: 2010-10-19 12:55 UTC (History)
1 user (show)

See Also:
Issue Type: TASK
Exception Reporter:

Attachments
Suggested change (5.50 KB, patch)
2008-07-09 15:23 UTC, Pavel Flaska 		Details | Diff
Unit test (3.23 KB, text/plain)
2008-07-14 12:33 UTC, Pavel Flaska 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Flaska 2008-07-09 14:34:15 UTC 
Currently, we do not have any facility which allows automated simple operation measuring from unit tests. We plan to write unit test to prevent performance 
regression. It seems it is enough to add special handler (similar to InstancesHandler) for time mesuring to netbeans junit extension module (Log.class).
Comment 1 Pavel Flaska 2008-07-09 15:23:36 UTC 
Created attachment 64185 [details]
Suggested change
Comment 2 Jesse Glick 2008-07-09 17:41:24 UTC 
[JG01] Link to

http://wiki.netbeans.org/FitnessViaTimersCounters

not the old

http://wiki.netbeans.org/wiki/view/FitnessViaTimersCounters


[JG02] It's unclear from both the Javadoc and the impl what these methods actually do. What is the msg param for? (It is
unused.) Why is Object[] result computed but never used? What does time measurement have to do with logging, anyway? I
think you need to look over both the implementation and the Javadoc more carefully and describe precisely what this
infrastructure is for and how it works.
Comment 3 Jaroslav Tulach 2008-07-11 07:13:18 UTC 
Y01 Write a test.
Comment 4 Pavel Flaska 2008-07-14 12:33:19 UTC 
Created attachment 64422 [details]
Unit test
Comment 5 Marian Petras 2008-07-14 15:49:24 UTC 
Changed the component/subcomponent to "xtest/nbjunit".
Comment 6 Jaroslav Tulach 2010-10-19 12:55:30 UTC 
Somehow forgotten and I am not sure if this change is needed right now. Let's leave it in bugzilla for now.