mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-24 09:25:31 +02:00
Code Issues Wiki Activity

[195830] RSE performs unnecessary remote list commands

Browse source
This commit is contained in:
Javier Montalvo Orus 2007-08-01 17:09:56 +00:00
parent 4f92ce3fba
commit 757b183b1f
1 changed files with 34 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
rse/plugins/org.eclipse.rse.services.files.ftp/src/org/eclipse/rse/internal/services/files/ftp/FTPService.java
View file

@ -52,6 +52,7 @@
* Javier Montalvo Orus (Symbian) - [197758] Unix symbolic links are not classified as file vs. folder
* Javier Montalvo Orus (Symbian) - [198182] FTP export problem: RSEF8057E: Error occurred while exporting FILENAME: Operation failed. File system input or output error
* Javier Montalvo Orus (Symbian) - [192610] EFS operations on an FTP connection make Eclipse freeze
* Javier Montalvo Orus (Symbian) - [195830] RSE performs unnecessary remote list commands
********************************************************************************/
package org.eclipse.rse.internal.services.files.ftp;
@ -67,6 +68,7 @@ import java.io.InputStream;
import java.io.OutputStream;
import java.text.MessageFormat;
import java.util.ArrayList;
import java.util.Hashtable;
import java.util.List;
import org.apache.commons.net.ftp.FTP;
@ -115,6 +117,12 @@ public class FTPService extends AbstractFileService implements IFileService, IFT
private IFTPClientConfigFactory _entryParserFactory;
private IFTPClientConfigProxy _clientConfigProxy;
//workaround to access FTPHostFile objects previously retrieved from the server
//to avoid accessing the remote target when not necessary (bug 195830)
//In the future, it would be better that the IHostFile object were passed from
//the upper layer instead of the folder and file name.
private Hashtable fileMap = new Hashtable();
private class FTPBufferedInputStream extends BufferedInputStream {
private FTPClient client;
@ -540,6 +548,12 @@ public class FTPService extends AbstractFileService implements IFileService, IFT
_ftpLoggingOutputStream.write(System.getProperty("line.separator").getBytes()); //$NON-NLS-1$
for (int i = 0; i < results.size(); i++) {
FTPHostFile file = (FTPHostFile)results.get(i);
fileMap.put(file.getAbsolutePath(), file);
}
}
catch (Exception e)
{
@ -695,7 +709,12 @@ public class FTPService extends AbstractFileService implements IFileService, IFT
}
}
IHostFile remoteHostFile = getFile(remoteParent,remoteFile,null);
IHostFile remoteHostFile = (IHostFile)fileMap.get(remoteParent+getSeparator()+remoteFile);
if(remoteHostFile == null)
{
remoteHostFile = getFile(remoteParent,remoteFile,null);
}
if(_commandMutex.waitForLock(monitor, Long.MAX_VALUE))
{
@ -813,7 +832,14 @@ public class FTPService extends AbstractFileService implements IFileService, IFT
progressMonitor.init(FTPServiceResources.FTP_File_Service_Deleting_Task+fileName, 1);
boolean isFile = getFile(remoteParent,fileName,null).isFile();
IHostFile file = (IHostFile)fileMap.get(remoteParent+getSeparator()+fileName);
if(file == null)
{
file = getFile(remoteParent,fileName,null);
}
boolean isFile = file.isFile();
try {
hasSucceeded = FTPReply.isPositiveCompletion(ftpClient.cwd(remoteParent));
@ -1161,7 +1187,12 @@ public class FTPService extends AbstractFileService implements IFileService, IFT
boolean result = false;
int permissions = 0;
FTPHostFile file = (FTPHostFile)getFile(parent,name, monitor);
FTPHostFile file = (FTPHostFile)fileMap.get(parent+getSeparator()+name);
if(file == null)
{
file =(FTPHostFile)getFile(parent,name, monitor);
}
int userPermissions = file.getUserPermissions();
int groupPermissions = file.getGroupPermissions();