[#JDO-410] AggregateResult queries non-existent Employee.manager.salary field

JDO

AggregateResult queries non-existent Employee.manager.salary field

Created: 30/Aug/06 12:23 AM Updated: 03/Oct/06 10:03 PM

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-410.patch	2006-09-10 06:50 PM	Michael Bouschen	4 kB

Resolution Date:

14/Sep/06 09:38 PM

Description

« Hide

In 6 different places in the AggregateResult test, a query is issued against FullTimeEmployee and returns an aggregate on "manager.salary". However, the "manager" field is of type Employee, and the "salary" field is only defined in the FullTimeEmployee subclass of Employee, thus rendering the aggregate statement invalid.

The solution is to replace all instances of "manager.salary" with a clause that casts "manager" down to "FullTimeEmployee". E.g.: "(((FullTimeEmployee)manager).salary)"

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Michael Bouschen made changes - 01/Sep/06 03:54 PM

Field	Original Value	New Value
Fix Version/s		JDO 2 maintenance release 1 [ 12310923 ]
Assignee		Michael Bouschen [ mbo ]

[ Permlink | « Hide ]

Michael Bouschen added a comment - 10/Sep/06 06:50 PM

The attached patch for review implements the proposed fix.

Michael Bouschen made changes - 10/Sep/06 06:50 PM

Attachment

JDO-410.patch [ 12340531 ]

[ Permlink | « Hide ]

Michael Bouschen added a comment - 14/Sep/06 09:38 PM

Checked in the patch (see revision 443484).

Michael Bouschen made changes - 14/Sep/06 09:38 PM

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

Repository	Revision	Date	User	Message
ASF	#443484	Thu Sep 14 21:40:58 UTC 2006	mbo	~~JDO-410~~: AggregateResult queries non-existent Employee.manager.salary field
				Files Changed
				MODIFY /db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/query/result/AggregateResult.java

Repository	Revision	Date	User	Message
ASF	#452652	Tue Oct 03 22:02:35 UTC 2006	mbo	~~JDO-410~~: merge changes from trunk into branch 2.0.1
				Files Changed
				MODIFY /db/jdo/branches/2.0.1/tck20/src/java/org/apache/jdo/tck/query/result/AggregateResult.java

[ Permlink | « Hide ]

Michael Bouschen added a comment - 03/Oct/06 10:03 PM

Merged changed from trunk into branch 2.0.1 (see revision 452652).

Michael Bouschen made changes - 03/Oct/06 10:03 PM

Fix Version/s

JDO 2.0 TCK challenge fixes [ 12312093 ]