jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jukka Zitting" <jukka.zitt...@gmail.com>
Subject Re: Refactoring of the backupTool
Date Tue, 05 Sep 2006 10:05:13 GMT
Hi,

On 9/4/06, Nicolas <ntoper@gmail.com> wrote:
> I am following our IM chat where you explained me a few things I didn't
> understand at first. I will summarize it here on the ML.

Thanks!

> - Actually between RestoreBatchedItemOperations and BatchedItemOperations, I
> only have deleted the "recursively add 'auto-create' child nodes defined in
> node type" part. This way, I only create node. This way we only create node
> and so don't check for protection (since a node to be created is not yet
> protected :p). I could add some composition to this, but only to this part
> seems overkill. I don't really see what I can do on this. What do you think?

If it's just a single feature then adding a flag for
enabling/disabling it could also solve the need.

> - NodeVersionHistoriesUpdatableStateManager and UpdatableStateItemManager.
> Actually, I created one new UISM because I couldn't understand how the
> EventStateCollectionFactory worked (I tried null and to instantiate one but
> it didn't work), therefore couldn't reuse the LocalItemStateManager. If
> someone can explain it to me, I might be able to delete this class.

The ESCF is used for sending observation events. For example the
VersionManagerImpl contains the escFactory instance that you should be
able to use.

BR,

Jukka Zitting

-- 
Yukatan - http://yukatan.fi/ - info@yukatan.fi
Software craftsmanship, JCR consulting, and Java development

Mime
View raw message