[ http://issues.apache.org/jira/browse/JCR-556?page=all ]

Nicolas Toper updated JCR-556:
------------------------------

    Attachment: patch-backup-040906.txt

Hi,

Here are some refactoring to the backup tool. This goes in the contrib package 
for the backup tool. It is mainly clean up of the code (remove some unused 
variables and comments)

There is one issue remaining with CRC check in the zip. Maybe you can help me: 
I calculate a CRC for each entry. With the debugger, I see it there, but when I 
open the file in read mode, it is not here anymore. Does anyone know why? It is 
important I believe.

Next steps are for me to code two patches:

1- To integrate Jukka's ideas

2- To implement those changes in this code (only a few lines will be changed)

BR
Nico

> Refactoring of the BackupTool
> -----------------------------
>
>                 Key: JCR-556
>                 URL: http://issues.apache.org/jira/browse/JCR-556
>             Project: Jackrabbit
>          Issue Type: Improvement
>            Reporter: Nicolas Toper
>            Priority: Minor
>         Attachments: patch-backup-040906.txt, 
> patch-jr-010906-NodeVersionHistoriesUpdatableStateManager.txt, 
> patch-jr-010906-PropInfo.txt, patch-jr-010906-RepositoryImpl.txt, 
> patch-jr-010906-SysViewImporter.txt, patch-jr-010906-VersionManagerImpl.txt
>
>
> The BackupTool has still some refactoring to perform. I will propose the 
> patch on this new issue since we cannot change the JCR-442 (Google wants to 
> see a place where all the Google SoC work is).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to