[#JDO-415] DataStoreConnection assumes datastore transactions are being used

JDO

DataStoreConnection assumes datastore transactions are being used

Created: 30/Aug/06 08:59 AM Updated: 05/Oct/06 02:49 AM

Return to search

Component/s:

tck2

Affects Version/s:

JDO 2 final

Fix Version/s:

JDO 2 maintenance release 1, JDO 2 TCK challenge fixes

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	JDO-415.patch	2006-10-03 10:53 PM	Craig Russell	2 kB

Resolution Date:

05/Oct/06 02:45 AM

Description

« Hide

org.apache.jdo.tck.api.persistencemanager.DataStoreConnection assumes that the connection returned from getDataStoreConnection will be re-used once a JDO transaction is started, but that assertion is only valid when using datastore transactions, and the test case does not ensure that a datastore transaction is being used.

The solution is to call "getPM().currentTransaction().setOptimistic(false);" just before "getPM().currentTransaction().begin();" in the testDataStoreConnection() test case.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#453089	Thu Oct 05 02:45:37 UTC 2006	clr	~~JDO-415~~ Three problems with the original test case are fixed with this patch. 1. As the bug noted, the DataStoreConnection is acquired outside a transaction, and there is no guarantee that the DataStoreConnection is even valid after beginning a transaction. The code now acquires the DataStoreConnection only after beginning a transaction. 2. If the implementation supports Optimistic transactions, then the test should be repeated for both optimistic and datastore transactions. 3. The transaction is never committed. This is fixed by putting the DataStoreConnection close and the transaction.commit into the finally block.
				Files Changed
				MODIFY /db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/api/persistencemanager/DataStoreConnection.java

Repository	Revision	Date	User	Message
ASF	#453090	Thu Oct 05 02:48:26 UTC 2006	clr	~~JDO-415~~
				Files Changed
				MODIFY /db/jdo/branches/2.0.1/tck20/src/java/org/apache/jdo/tck/api/persistencemanager/DataStoreConnection.java