FTP very slow

Afternoon All

Not quite sure if this is the right forum section...

I need to implement FTP transfer in a program I support. First time playing with FTP in Java so I knocked up a quick program see what was needed and got some disappoiting results.

If I use FileZilla or the DOS bases FTP command to transfer the 2.37MB test file, it takes about 6 seconds. Using either Commons Net of sun.net.ftp I get 20-30 seconds.

Can anyone shed any light on why there is such a difference?

Apologies for the huge post, but it wouldn't let me attach .java, .txt or .zip files!

Tom

Commons Appraoch

FTPClient ftpClient;
        SimpleDateFormat sdf = new SimpleDateFormat("HH:mm:ss");
        try
        {
            ftpClient = new FTPClient();
            ftpClient.connect(url);
            ftpClient.login(user, pswd);
            ftpClient.enterLocalPassiveMode();
            ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
            if (FTPReply.isPositiveCompletion(ftpClient.getReplyCode()))
            {
                System.out.println("Connected");
                // upload
                File f = new File("d://FTP.pdf");
                if (f.exists())
                {
                    FileInputStream fis = new FileInputStream(f);
                    System.out.println("Start Upload:" + sdf.format(new Date()));
                    ftpClient.storeFile(f.getName(), fis);
                    System.out.println("Finsih Upload:" + sdf.format(new Date()));
                }
            }
            else
            {
                System.out.println("Connection failed");
                ftpClient.disconnect();
            }

}
        catch (SocketException se)
        {
            System.out.println(se.getMessage());
        }
        catch (IOException ioe)
        {
            System.out.println(ioe.getMessage());
        }
        catch (Exception e)
        {
            System.out.println(e.getMessage());
        }

Sun Approach

/** Get the present working directory */
	public String pwd() throws IOException {
		issueCommand("PWD");
		if (isValidResponse()) {
			String response = getResponseString().substring(4).trim();
			if (response.startsWith("\""))
				response = response.substring(1);
			if (response.endsWith("\""))
				response = response.substring(0, response.length() - 1);
			return response;
		} else {
			return "";
		}
	}

/** Go up one directory */
	public boolean cdup() throws IOException {
		issueCommand("CDUP");
		return isValidResponse();
	}

/** Create a new directory */
	public boolean mkdir (String newDir) throws IOException {
		issueCommand("MKDIR " + newDir);
		return isValidResponse();
	}

/** Delete a remote file */
	public boolean deleteFile (String fileName) throws IOException {
		issueCommand("DELE " + fileName);
		return isValidResponse();
	}

/** Get the results of the LIST command as a Vector of Strings.
	  * Because there's no standard format for the results of a LIST
	  * command, it's hard to tell what resulting data will look like.
	  * Just be aware that different servers have different ways of
	  * returning your LIST data. */
	public Vector listRaw() throws IOException {
		String fileName;
		Vector ftpList = new Vector();
		BufferedReader reader = new BufferedReader(new InputStreamReader(list()));
		while ((fileName = reader.readLine()) != null) {
			ftpList.add(fileName);
		}
		return ftpList;
	}

/** Get the response code from the last command that was sent */
	public int getResponseCode() throws NumberFormatException {
		return Integer.parseInt(getResponseString().substring(0, 3));
	}

/** Return true if the last response code was in the 200 range,
	  * false otherwise */
	public boolean isValidResponse() {
		try {
			int respCode = getResponseCode();
			return (respCode  >= 200 && respCode < 300);
		} catch (Exception e) {
			return false;
		}
	}

/** Send a raw FTP command to the server. You can get the response
	  * by calling getResponseString (which returns the entire response as a
	  * single String) or getResponseStrings (which returns the response
	  * as a Vector). */
	public int issueRawCommand (String command) throws IOException {
		return issueCommand(command);
	}

/** Download a file from the server, and save it to the specified local file */
	public boolean downloadFile (String serverFile, String localFile) throws IOException {
		int i = 0;
		byte[] bytesIn = new byte[1024];
		BufferedInputStream in = new BufferedInputStream(get(serverFile));
		FileOutputStream out = new FileOutputStream(localFile);
		while ((i = in.read(bytesIn)) >= 0) {
			out.write(bytesIn, 0, i);
		}
		out.close();
		return true;
	}

/** Upload a file to the server */
	public boolean uploadFile (String localFile, String serverFile) throws IOException {
		int i = 0;
		byte[] bytesIn = new byte[1024];
		FileInputStream in = new FileInputStream(localFile);
		BufferedOutputStream out = new BufferedOutputStream(put(serverFile));
		while ((i = in.read(bytesIn)) >= 0) {
			out.write(bytesIn, 0, i);
		}
		in.close();
		out.close();
		return true;
	}

Having installed Wireshark I think I might have found something...

With Wireshark I was able to see that when using FileZilla each dollop of data sent was 1360 bytes, however checked the Java approaches the dollops ranged from 688 to 1360 bytes.

I tried adding a buffered read, just in case...

It made no difference.

Tom