[#OPENJPA-118] AutoDetach property has no effect

OpenJPA

AutoDetach property has no effect

Created: 31/Jan/07 04:05 PM Updated: 01/Mar/07 02:13 AM

Return to search

Component/s:

lib

Affects Version/s:

None

Fix Version/s:

0.9.7

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	TestFixesForOpenJPA-118.zip	2007-01-31 04:10 PM	David Ezzio	18 kB

Environment:

Java SE

Resolution Date:

31/Jan/07 10:27 PM

Description

« Hide

Setting the openjpa.AutoDetach property to the string "commit, close, nontx-read" has no effect.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

David Ezzio added a comment - 31/Jan/07 04:10 PM

Tests and fixes for the bug. Not sure if the tests are in the right module. Perhaps they should be in openjpa-persistence instead of openjpa-persistence-jdbc. The fixes also clean up the error message when the property has invalid values to conform to the documented settings.

David Ezzio made changes - 31/Jan/07 04:10 PM

Field	Original Value	New Value
Attachment		TestFixesForOpenJPA-118.zip [ 12350043 ]

[ Permlink | « Hide ]

Patrick Linskey added a comment - 31/Jan/07 05:40 PM

Sadly, due to some maven environment issues, all tests that need to communicate with a database need to be in openjpa-persistence-jdbc.

Repository	Revision	Date	User	Message
ASF	#502022	Wed Jan 31 22:27:29 UTC 2007	mprudhom	~~OPENJPA-118~~: Implemented patch provided by David Ezzio for broken openjpa.AutoDetach behavior
				Files Changed
				MODIFY /incubator/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java ADD /incubator/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/conf/TestAutoDetachProperty.java ADD /incubator/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/conf MODIFY /incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/StringListValue.java MODIFY /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/AutoDetachValue.java ADD /incubator/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/conf/TestBadAutoDetachProperty.java MODIFY /incubator/openjpa/trunk/openjpa-lib/src/main/resources/org/apache/openjpa/lib/conf/localizer.properties

[ Permlink | « Hide ]

Marc Prud'hommeaux added a comment - 31/Jan/07 10:27 PM

I've corrected a couple minior problems with the patch (no ASF license header, files in DOS format, non-standard indentation & linebreaks) and committed. Tests pass, closing issue.

Marc Prud'hommeaux made changes - 31/Jan/07 10:27 PM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Patrick Linskey made changes - 01/Mar/07 02:13 AM

Fix Version/s

0.9.7 [ 12312340 ]