|
Home > Archive > Apache JDO Project > June 2005 > Additional dependency from r11 runtime to r11 query
You are viewing an archived Text-only version of the thread.
To view this thread in it's original format and/or if you want to reply to
this thread please [click here]
| Author |
Additional dependency from r11 runtime to r11 query
|
|
| Michael Watzek 2005-06-06, 7:49 am |
| Hi,
I encountered an additional dependency (which I have missed before)
from r11 runtime to r11 query: org.apache.jdo.store.StoreManager and
org.apache.jdo.store.StoreManagerImpl use QueryResult, and
QueryResultHelper, BasicQueryResult.
Breaking out r11 into core20, enhancer20, runtime20, query20, and
fostore20 as decided means that we would have circular dependencies
between query20 and runtime20.
Preventing these circular dependencies in XXX20 projects I have the
following proposal:
1) Repackaging of the following classes, interfaces, and resources in r11:
org.apache.jdo.jdoql.QueryResult -> org.apache.jdo.query.QueryResult
org.apache.jdo.jdoql.QueryResultHelper ->
org.apache.jdo.query.QueryResultHelper
org.apache.jdo.jdoql.QueryResultIterator ->
org.apache.jdo.query.QueryResultIterator
org.apache.jdo.jdoql.BasicQueryResult ->
org.apache.jdo.query.BasicQueryResult
org/apache/jdo/jdoql/Bundle.properties ->
org/apache/jdo/query/Bundle.properties
2) At the time we break out r11 into XXX20 projects we assign package
org.apache.jdo.query (and all of its classes/interfaces/resources) to
runtime20.
If you do not have any objections, I'll send the "svn diff" on Monday.
Thanks,
Michael
--
-------------------------------------------------------------------
Michael Watzek Tech@Spree Engineering GmbH
mailto:mwa.tech@spree.de Buelowstr. 66
Tel.: ++49/30/235 520 36 10783 Berlin - Germany
Fax.: ++49/30/217 520 12 http://www.spree.de/
-------------------------------------------------------------------
| |
| Michael Watzek 2005-06-06, 7:49 am |
| Hi,
attached you find the patch file (rio.path) containing the changes
described below. Please execute "patch -E -p0 < rio.patch" to apply the
patch to your workspace. Parameter "-E" will remove empty files.
If you decide to checkin the patch, please make sure that you add/remove
the files of the patch in subversion:
- "svn add src/java/org/apache/jdo/query"
- "svn remove src/java/org/apache/jdo/jdoql/Bundle.properties
src/java/org/apache/jdo/jdoql/QueryResultHelper.java
src/java/org/apache/jdo/jdoql/QueryResult.java
src/java/org/apache/jdo/jdoql/BasicQueryResult.java
src/java/org/apache/jdo/jdoql/QueryResultIterator.java"
Regards,
Michael
> Hi,
>
> I encountered an additional dependency (which I have missed before)
> from r11 runtime to r11 query: org.apache.jdo.store.StoreManager and
> org.apache.jdo.store.StoreManagerImpl use QueryResult, and
> QueryResultHelper, BasicQueryResult.
>
> Breaking out r11 into core20, enhancer20, runtime20, query20, and
> fostore20 as decided means that we would have circular dependencies
> between query20 and runtime20.
>
> Preventing these circular dependencies in XXX20 projects I have the
> following proposal:
>
> 1) Repackaging of the following classes, interfaces, and resources in r11:
>
> org.apache.jdo.jdoql.QueryResult -> org.apache.jdo.query.QueryResult
> org.apache.jdo.jdoql.QueryResultHelper ->
> org.apache.jdo.query.QueryResultHelper
> org.apache.jdo.jdoql.QueryResultIterator ->
> org.apache.jdo.query.QueryResultIterator
> org.apache.jdo.jdoql.BasicQueryResult ->
> org.apache.jdo.query.BasicQueryResult
> org/apache/jdo/jdoql/Bundle.properties ->
> org/apache/jdo/query/Bundle.properties
>
> 2) At the time we break out r11 into XXX20 projects we assign package
> org.apache.jdo.query (and all of its classes/interfaces/resources) to
> runtime20.
>
> If you do not have any objections, I'll send the "svn diff" on Monday.
>
> Thanks,
> Michael
--
-------------------------------------------------------------------
Michael Watzek Tech@Spree Engineering GmbH
mailto:mwa.tech@spree.de Buelowstr. 66
Tel.: ++49/30/235 520 36 10783 Berlin - Germany
Fax.: ++49/30/217 520 12 http://www.spree.de/
-------------------------------------------------------------------
| |
| Michael Bouschen 2005-06-06, 7:49 am |
| Hi Michael,
thanks a lot for the patch. I checked in the changes
(http://svn.apache.org/viewcvs?rev=170666&view=rev).
Regards Michael
> Hi,
>
> attached you find the patch file (rio.path) containing the changes
> described below. Please execute "patch -E -p0 < rio.patch" to apply
> the patch to your workspace. Parameter "-E" will remove empty files.
>
> If you decide to checkin the patch, please make sure that you
> add/remove the files of the patch in subversion:
>
> - "svn add src/java/org/apache/jdo/query"
> - "svn remove src/java/org/apache/jdo/jdoql/Bundle.properties
> src/java/org/apache/jdo/jdoql/QueryResultHelper.java
> src/java/org/apache/jdo/jdoql/QueryResult.java
> src/java/org/apache/jdo/jdoql/BasicQueryResult.java
> src/java/org/apache/jdo/jdoql/QueryResultIterator.java"
>
> Regards,
> Michael
>
>
--
Michael Bouschen Tech@Spree Engineering GmbH
mailto:mbo.tech@spree.de http://www.tech.spree.de/
Tel.:++49/30/235 520-33 Buelowstr. 66
Fax.:++49/30/2175 2012 D-10783 Berlin
|
|
|
|
|