Issue Details (XML | Word | Printable)

Key: JDO-36
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Michael Bouschen
Reporter: Michael Watzek
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
JDO

Implement JDOQLQueryFactory proposal

Created: 05/May/05 01:09 AM   Updated: 01/Nov/05 05:33 PM
Return to search
Component/s: ri11
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

Resolution Date: 06/May/05 05:45 AM


 Description  « Hide
There is a mail thread "Dependencies between ri11 runtime and r11 query" on jdo-dev that requests some preparation work before we start to solve the dependencies between runtime and query:

We agreed on introducing a JDOQLQueryFactory interface declaring "newQuery" methods and a "newQueryTree" method. This interface will be placed in package "org.apache.jdo.jdoql". An implementation class of this interface (JDOQLQueryFactoryImpl) will be placed in package "org.apache.jdo.impl.jdoql".

An instance of JDOQLQueryFactory will be kept in FOStorePMF. This instance will be created by "Class.newInstance".

We introduce a new class FOStorePM extending PersistenceManagerImpl. This class implements all "newQuery" methods delegating to JDOQLQueryFactoryImpl. Class PersistenceManagerImpl will be made abstract removing all "newQuery" methods.


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Repository Revision Date User Message
ASF #168400 Thu May 05 20:38:26 UTC 2005 mbo JDO-36 Implement JDOQLQueryFactory proposal. Code provided by Michael Watzek
Files Changed
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/JDOQLQueryFactory.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/jdoql/JDOQLQueryFactoryImpl.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/pm/PersistenceManagerFactoryImpl.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/pm/PersistenceManagerImpl.java
MODIFY /incubator/jdo/trunk/ri11/test/java/org/apache/jdo/test/query/PositiveTest.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/fostore/Bundle.properties
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/fostore/FOStorePMF.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/fostore/FOStorePM.java
MODIFY /incubator/jdo/trunk/ri11/test/java/org/apache/jdo/test/query/NegativeTest.java

Michael Bouschen made changes - 06/May/05 05:03 AM
Field Original Value New Value
Assignee Michael Bouschen [ mbo ]
Michael Bouschen added a comment - 06/May/05 05:45 AM
Michael Watzek implemented the JDOQLQueryFactory proposal as described above. It passes all the ri11 tests.

Michael Bouschen made changes - 06/May/05 05:45 AM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Repository Revision Date User Message
ASF #170666 Tue May 17 21:17:50 UTC 2005 mbo JDO-36: Implement JDOQLQueryFactory proposal; moved query interfaces to a new package
Files Changed
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/store/StoreManagerImpl.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/store/StoreManager.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/jdoql/QueryImpl.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query/Bundle.properties
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query/QueryResult.java
DEL /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/BasicQueryResult.java
DEL /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/Bundle.properties
DEL /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/QueryResultHelper.java
DEL /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/QueryResultIterator.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/jdoql/QueryResultHelperImpl.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query/QueryResultHelper.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query/QueryResultIterator.java
ADD /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/query/BasicQueryResult.java
MODIFY /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/impl/fostore/FOStoreStoreManager.java
DEL /incubator/jdo/trunk/ri11/src/java/org/apache/jdo/jdoql/QueryResult.java

Michael Bouschen made changes - 01/Nov/05 05:33 PM
Status Resolved [ 5 ] Closed [ 6 ]