Mailing-List: contact dev-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@jackrabbit.apache.org
Received-SPF: pass (athena.apache.org: domain of jukka.zitting@gmail.com
 designates 64.233.162.226 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=hm9uVfKq3mOZf06/4MfUiIt5G9l9cnU/ZjvCaKK2RkLA7WeyrAhmONlUQZFxgc3oikXJYUk3P3OOdIs6bZSCvKKDconBqC0OSbTZsT4P7Qp6WFlK2ijm4Nyx5KYi0ruevApac54BX3L9GZYgQ+xCNMswvzFt5tx8kmVruEhM97s=
Message-ID: <510143ac0708280855n42863913j8ed7ce51f797ecef@mail.gmail.com>
Date: Tue, 28 Aug 2007 18:55:14 +0300
From: "Jukka Zitting" <jukka.zitting@gmail.com>
To: dev@jackrabbit.apache.org
Subject: Re: Database connections & queries
In-Reply-To: <510143ac0708241338q36f1f329g6e1d279ec6f20b3a@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: 
 <34B0CDC6176518459F3A96E8C09196B8062DB767@darth-vader.nijmegen.gx.nl>
	 <510143ac0708241338q36f1f329g6e1d279ec6f20b3a@mail.gmail.com>

Hi,

On 8/24/07, Jukka Zitting <jukka.zitting@gmail.com> wrote:
> On 8/22/07, Martijn Hendriks <martijnh@gx.nl> wrote:
> > The current implementation of the LazyScoreNodeIterator is such to
> > ignore all RepositoryExceptions that might be thrown while loading the
> > nodes in the result set, as required by the javax.jcr.NodeIterator spec.
>
> I think the correct behaviour would be for the node iterator to throw
> an exception (unfortunately I guess only a RuntimeException would work
> here) instead of ignoring such internal errors.

A thought just crossed my mind... As mentioned in another thread a few
days ago, we could make NodeImpl to load the underlying node states
only on demand as long as we have just the node ID, parent ID, and
name of the node available. This way we could "delay" the exception to
a method like getProperty() that is allowed to throw appropriate
exceptions.

Besides, such a solution would most likely give a nice speedup to
clients that just want to get the names or paths of nodes that match a
query...

BR,

Jukka Zitting