Re: [jbosscache-dev] GravitateDataCommand finds invalid "phantom" nodes

Thanks for the patch; that makes a lot of sense. Go ahead and check it in to trunk. Regarding the creation of nodes for deletion, well that's pessimistic locking for you. Unfortunately the lock is encapsulated in the node so you can't acquire the lock until the node is created. (Which is partially why the pessimistic lock design sucks). This is created in PessimisticNodeBasedLockManager (line 145). At this stage we already know if we are just creating a dummy node for deletion, so perhaps we could pass this as a flag into the constructor of a PessimisticUnversionedNode, and this could acquire the lock on creation. Hmm, need to think about this.