db-jdo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andy Jefferson (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JDO-405) recursion-depth - fetching the whole graph with 0 or -1?
Date Wed, 09 Aug 2006 09:07:16 GMT
    [ http://issues.apache.org/jira/browse/JDO-405?page=comments#action_12426877 ] 
            
Andy Jefferson commented on JDO-405:
------------------------------------

AFAIK the p127 comment is erroneous and is a cut-paste. Only one value is needed for representing
"no limit" and this was -1 with "fetch-depth" (before the renaming) and is what 12.7.4 has
(and would be consistent with what values maxFetchDepth takes).

I would add that page 129 has an example that uses "fetch-depth" which was removed/renamed,
so this needs fixing.

I don't see a reason to change the default we agreed some time back. Default of maxFetchDepth
of 1 and recursion-depth of 1 are both consistent with fetching just the immediate objects.

> recursion-depth - fetching the whole graph with 0 or -1?
> --------------------------------------------------------
>
>                 Key: JDO-405
>                 URL: http://issues.apache.org/jira/browse/JDO-405
>             Project: JDO
>          Issue Type: Bug
>    Affects Versions: JDO 2 final
>            Reporter: Ilan Kirsh
>
> In page 124: "A value of -1 means that the recursion-depth is not limited by traversing
this field."
> In page 127: "A recursion-depth of 0 will fetch the whole graph of instances reachable
from this field"
> Is there a difference between these two?
> In addition, I think that -1 / 0 as a default might be better than 1 it terms of performance,
because tracking such cycles has some overhead, which we might not want to pay for most fields
in which this feature is not needed (and MaxFetchDepth is sufficient).

-- 
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

        

Mime
View raw message