Patch for external project it.could.webdav

• Key: HADOOP-505
• Type: Task
• Status: Closed
• Resolution: Invalid
• Priority: Major
• Assignee: Sameer Paranjpye
• Reporter: Michel Tourn

Description

Context:
http://issues.apache.org/jira/browse/HADOOP-496
http://could.it/main/a-simple-approach-to-webdav.html

This patch applies to an external project that Hadoop would use as a library.
You can obtain the baseline code:
svn co --revision {"1 Sep 2006 00:00:00 GMT"} http://could.it/repo/webdav/head/
Then apply the attached patch.

Patch details below.
---------------------
Changes in the patch are:

Filehandle leaks prevented it from working on Windows. (in.close() in 3 places)
Filehandle leaks would eventually crash server on Linux.

Now it passes the litmus compliance tests, in both original and HDFS mode.
Mostly had to refine the HTTP status codes.

Decoupled COPY and MOVE
This allows an implementation of MOVE other than COPY-and-delete

There is no new "abstraction code" to support a distributed filesystem backend.
Only: one change to make a java.io.File reference work when it represents a
non-Windows path on Windows: (unixAbsolute = rootPath.startsWith("/"))

The factory code via servlet properties was extended to allow loading by name:
repositoryClass="it.could.webdav.DAVRepository"
repositoryClass="it.could.webdav.XMLRepository"
repositoryClass="org.apache.hadoop.dfs.webdav.HDFSRepository"

4 classes are subclassed for the alternative implementation:
A few private fields were made protected.
Some constructor logic moved to protected init() method-s so it can be overriden.

Comments

[Pier Fumagalli]

Patch applied onto HEAD revision at http://could.it/repo/webdav/head/

Will be availble in 0.5 at http://could.it/repo/webdav/tags/0.5/ once the new release is rolled!

[Enis Soztutar]

Marking this issue as invalid, since we will stick to Jackrabbit's webdav API's for webdav.