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 32345 - CVS Update does too much unnecessary refreshing
Summary: CVS Update does too much unnecessary refreshing
Status: RESOLVED WONTFIX
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcscore (show other bugs)
Version: 5.x
Hardware: All Windows XP
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: PERFORMANCE
: 47958 (view as bug list)
Depends on: 32089 35705
Blocks:
  Show dependency tree
 
Reported: 2003-03-26 10:22 UTC by Martin Entlicher
Modified: 2009-05-15 11:58 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Graph of CPU usage during "CVS | Update" command. (202.28 KB, image/jpeg)
2003-04-28 11:08 UTC, Jiri Kovalsky
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Entlicher 2003-03-26 10:22:28 UTC
This problem was initially submitted in issue
#32065 by Andrew Gaydenko:

"After VCS update command finished (judging by the
status string), the IDE eats 100% CPU during some
period. This period grows during working with the
IDE from 4-5 sec to 15-20 sec."
Comment 1 Martin Entlicher 2003-03-26 10:39:21 UTC
Scheduling for 4.0, this is too much risky changes for NB 3.5 in this
stage.
Comment 2 andrew 2003-03-27 22:54:38 UTC
Martin,

I must define more accurately _current_ time frames
(after multiple changing in trunk :-). Update command
(win 2000 CVS profile) has 3 phases:

1. with not zero net traffic - [send+receive] ~ 15 sec,
2. HDD activity - ~7 sec (it doesn't depends on the IDE,
   I think),  CVS session is closed after this phase,
3. CPU 100% eating - ~18-20 sec :-(, and (this is the
   difference) only after this phase status string 
   reports updating was done. _No_ any more CPU eating now
   after status string report!

Last phase begins just after "Kill" button in UPDATE output
panel is switched to "Close".

Must I supply threads dumps for last phase?

P.S. Sorry, I'm not english-speaker. Is there more suitable
     term for "CPU eating" phrase?


Comment 3 dmladek 2003-04-25 17:50:53 UTC
Hi Andrew,

ThreadDumps are always valuaable, so go ahead and attach tmem, please.

BTW: Most of us aren't English native speakers too:-)
Main thing is to be able communicate and understand
Comment 4 andrew 2003-04-25 21:49:13 UTC
Daniel, Martin,

During last month many related issues were fixed.
From my point of view, the issue may be closed -
I don't notice cpu eating period growth.
Comment 5 Martin Entlicher 2003-04-28 10:21:00 UTC
I've made several various optimizations of status update after all
commands and update after checkout/update commands. However there
might be some more optimizations possible. QA please let me know
whether the current state is acceptable or whether it needs some more
tunning.
Comment 6 Jiri Kovalsky 2003-04-28 11:07:28 UTC
As for what I experienced during my small test I would say that
behaviour is rather same. I am attaching graph of CPU usage (5 seconds
grid size). It complies what Andrew described. There is certain period
of 100% CPU consumption after files are updated physically. Can this
be avoided ? I used  NetBeans 3.5 beta build #200304222350.

To Andrew: I saw that many native speakers use words "CPU consumption"
for what you tried to express. Anyway I am not native speaker too ... :-)
Comment 7 Jiri Kovalsky 2003-04-28 11:08:25 UTC
Created attachment 10175 [details]
Graph of CPU usage during "CVS | Update" command.
Comment 8 Martin Entlicher 2003-04-28 11:51:01 UTC
Thanks Jiri. So I will keep this issue opened and after we'll port VCS
modules into the new projects infrastructure in NB 4.0, I'll look at
this problem.
Comment 9 Martin Entlicher 2003-08-08 11:53:42 UTC
Adding PERFORMANCE keyword.
The file status cahcing system will be re-designed to improve
stability and performance (issue #32089). This should solve problems
of unnecessary refreshing of files. Also the update command should
have a better possibility to cooperate with the cache and update the
information only when necessary.
Comment 10 Martin Entlicher 2003-08-20 12:42:14 UTC
Adding dependency on issue #35512. findExistingResource() method
should improve the performance of FileObjects refreshing.
Comment 11 Martin Entlicher 2004-04-20 14:17:35 UTC
3.6 is out, re-scheduling for promotion D (4.0).
Adding dependency on issue #35705 rather then issue #35512.
Comment 12 Martin Entlicher 2004-10-19 18:41:20 UTC
*** Issue 47958 has been marked as a duplicate of this issue. ***
Comment 13 _ pkuzel 2005-01-11 13:55:08 UTC
FIXED by cache redesign
Comment 14 djfingers 2006-01-19 23:33:01 UTC
I installed the CVS module for VSS and it hangs on the command refresh.  It eats
up 100% of my CPU.   This happened in V4 as well so I upgraded to 5.
Comment 15 Jaroslav Tulach 2009-05-15 11:58:50 UTC
Perf team no longer cares about vcscore.