Review Request: [kio_sftp] Implement SlaveBase::copy to make file upload/download faster

Commit Hook null at kde.org
Sat Nov 12 04:48:34 GMT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103109/#review8131
-----------------------------------------------------------


This review has been submitted with commit 4acac3ad3fa7c3c212c4f9c123013daef8b07bec by Dawit Alemayehu to branch master.

- Commit Hook


On Nov. 10, 2011, 10:39 p.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103109/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2011, 10:39 p.m.)
> 
> 
> Review request for KDE Runtime and Andreas Schneider.
> 
> 
> Description
> -------
> 
> The attached patch:
> 
> - Implements SlaveBase::copy so that copying to and from the local filesystem is optimized. This makes the common use case of uploading and downloading to and from the local file system much much faster.
> - Sends the correct mime-type information when download a file from remote server.
> 
> 
> Diffs
> -----
> 
>   kioslave/sftp/kio_sftp.h c937e2e 
>   kioslave/sftp/kio_sftp.cpp adc918a 
>   kioslave/sftp/sftp.protocol 696647d 
> 
> Diff: http://git.reviewboard.kde.org/r/103109/diff/diff
> 
> 
> Testing
> -------
> 
> - Upload a large file from local file system to sftp server, cancel while still in progress and redo the upload.
> - Download a large file from sftp server to file system, cancel while still in progress and redo the download.
> - Repeat steps #1 and #2 with a list of files instead of just one large file.
> - Repeat the above steps after disabling "Mark partially uploaded files".
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111112/94ccb4b2/attachment.htm>


More information about the kde-core-devel mailing list