[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[Subclipse-dev] Subclipse "switch" very slow compared to "svn switch" (From users list)

From: Irving, Dave <dave.irving_at_logicacmg.com>
Date: 2006-04-10 15:44:34 CEST

Hi,
 
You may have seen my mail from the users list earlier commenting on how
a "switch" through subclipse is much slower than a "svn switch".
Alexander pointed me in the direction of the switch command, and after
checking out the source, I made the local at the end of this mail.
I simply took out the refreshLocal invocation after the call to
ISVNClientAdapter#switchToUrl.

My switch time dropped straight down to 1min 25 (just 5 seconds more
that a straight "svn switch ..." takes me from the command line.

However, Im sure there are all manner of nasty side effects of doing
this - which I will start to investigate this afternoon.
One thing someone may possibly be able to help me with though: Why do we
need to refresh the resource root for a switch, but not for an update?
(UpdateResourcesCommand doesn't trigger a refresh.....).
If decoration updates work for an update without a full refresh Im
confused as to why they wouldn't for a switch?

Anyways, for completeness, here's what I did:
 
Index:
C:/eclipse/3.2M4/eclipse/workspace/subclipse-core/src/org/tigris/subvers
ion/subclipse/core/commands/SwitchToUrlCommand.java
===================================================================

---
C:/eclipse/3.2M4/eclipse/workspace/subclipse-core/src/org/tigris/subvers
ion/subclipse/core/commands/SwitchToUrlCommand.java (revision 2164)
+++
C:/eclipse/3.2M4/eclipse/workspace/subclipse-core/src/org/tigris/subvers
ion/subclipse/core/commands/SwitchToUrlCommand.java (working copy)
@@ -48,11 +48,12 @@
             OperationManager.getInstance().beginOperation(svnClient,
new OperationProgressNotifyListener(subPm));
             File file = resource.getLocation().toFile();
             svnClient.switchToUrl(file, svnUrl, svnRevision, true);
-            try {
-                // Refresh the resource after merge
-                resource.refreshLocal(IResource.DEPTH_INFINITE, new
NullProgressMonitor());
-            } catch (CoreException e1) {
-            }
+//            try {
+//                // Refresh the resource after merge
+//
+//                resource.refreshLocal(IResource.DEPTH_INFINITE,
monitor);
+//            } catch (CoreException e1) {
+//            }
         } catch (SVNClientException e) {
             throw SVNException.wrapException(e);
         } finally {
Dave
This e-mail and any attachment is for authorised use by the intended recipient(s) only. It may contain proprietary material, confidential information and/or be subject to legal privilege. It should not be copied, disclosed to, retained or used by, any other party. If you are not an intended recipient then please promptly delete this e-mail and any attachment and all copies and inform the sender. Thank you.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subclipse.tigris.org
For additional commands, e-mail: dev-help@subclipse.tigris.org
Received on Mon Apr 10 15:45:31 2006

This is an archived mail posted to the Subclipse Dev mailing list.