[ http://issues.apache.org/jira/browse/JDO-244?page=all ]
Michael Bouschen resolved JDO-244:
----------------------------------
Resolution: Fixed
Good point Andy! Actually the generated SQL is correct.
I know what the problem is: the JDOQL query is not valid. It accesses a field
salary that is declared in a subclass of the candidate class. JPOX should catch
this, but this is a different problem and I filed a new JIRA issue for this
(see JDO-270). The entire inheritance hierarchy is mapped to single table, so
the salary column is null for a row that does not represent an instance of the
class that declares the salary field.
I changed the query to access a field declared in the candidate class.
> JPOX generates illegal SQL for having clauses using COUNT.
> ----------------------------------------------------------
>
> Key: JDO-244
> URL: http://issues.apache.org/jira/browse/JDO-244
> Project: JDO
> Type: Bug
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Erik Bengtson
> Attachments: JDO-244.patch
>
> JPOX generates illegal SQL for the query below. The having clause specifies
> an aggregate COUNT.
> 14:22:50,906 (main) DEBUG [org.apache.jdo.tck] - Executing API query: SELECT
> department, SUM(salary) FROM org.apache.jdo.tck.pc.company.Employee GROUP BY
> department HAVING COUNT(department.employees) > 0
> 14:22:51,031 (main) INFO [org.apache.jdo.tck] - Exception during setUp or
> runtest:
> javax.jdo.JDODataStoreException: Error executing JDOQL query "SELECT
> THIS.DEPARTMENT,SUM(THIS.SALARY) FROM applicationidentity0.PERSONS THIS LEFT
> OUTER JOIN applicationidentity0.DEPARTMENTS THIS_DEPARTMENT_EMPLOYEES ON
> THIS.DEPARTMENT = THIS_DEPARTMENT_EMPLOYEES.ID WHERE THIS.DISCRIMINATOR = ?
> OR THIS.DISCRIMINATOR = ? OR THIS.DISCRIMINATOR = ? GROUP BY THIS.DEPARTMENT
> HAVING COUNT() > 0" : Syntax error: Encountered ")" at line 1, column 324.
> ERROR 42X01: Syntax error: Encountered ")" at line 1, column 324.
> at org.apache.derby.iapi.error.StandardException.newException(Unknown
> Source)
> at org.apache.derby.impl.sql.compile.ParserImpl.parseStatement(Unknown
> Source)
> at org.apache.derby.impl.sql.GenericStatement.prepMinion(Unknown Source)
> at org.apache.derby.impl.sql.GenericStatement.prepare(Unknown Source)
> at
> org.apache.derby.impl.sql.conn.GenericLanguageConnectionContext.prepareInternalStatement(Unknown
> Source)
> at org.apache.derby.impl.jdbc.EmbedPreparedStatement.<init>(Unknown
> Source)
> at org.apache.derby.impl.jdbc.EmbedPreparedStatement20.<init>(Unknown
> Source)
> at org.apache.derby.impl.jdbc.EmbedPreparedStatement30.<init>(Unknown
> Source)
> at org.apache.derby.jdbc.Driver30.newEmbedPreparedStatement(Unknown
> Source)
> at org.apache.derby.impl.jdbc.EmbedConnection.prepareStatement(Unknown
> Source)
> at org.apache.derby.impl.jdbc.EmbedConnection.prepareStatement(Unknown
> Source)
> at
> com.mchange.v2.c3p0.impl.NewProxyConnection.prepareStatement(NewProxyConnection.java:190)
> at org.jpox.store.StatementText.prepareStatement(StatementText.java:199)
> at org.jpox.store.query.JDOQLQuery.performExecute(JDOQLQuery.java:678)
> at org.jpox.store.query.Query.executeWithMap(Query.java:966)
> at org.jpox.store.query.Query.executeWithArray(Query.java:939)
> at org.jpox.store.query.Query.execute(Query.java:862)
> at org.apache.jdo.tck.query.QueryTest.execute(QueryTest.java:1151)
> at org.apache.jdo.tck.query.QueryTest.execute(QueryTest.java:1029)
> at
> org.apache.jdo.tck.query.QueryTest.executeAPIQuery(QueryTest.java:966)
> at
> org.apache.jdo.tck.query.QueryTest.executeAPIQuery(QueryTest.java:946)
> at org.apache.jdo.tck.query.result.Having.testPositive(Having.java:110)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
> at java.lang.reflect.Method.invoke(Method.java:324)
> at junit.framework.TestCase.runTest(TestCase.java:154)
> at org.apache.jdo.tck.JDO_Test.runBare(JDO_Test.java:204)
> at junit.framework.TestResult$1.protect(TestResult.java:106)
> at junit.framework.TestResult.runProtected(TestResult.java:124)
> at junit.framework.TestResult.run(TestResult.java:109)
> at junit.framework.TestCase.run(TestCase.java:118)
> at junit.framework.TestSuite.runTest(TestSuite.java:208)
> at junit.framework.TestSuite.run(TestSuite.java:203)
> at junit.framework.TestSuite.runTest(TestSuite.java:208)
> at junit.framework.TestSuite.run(TestSuite.java:203)
> at junit.textui.TestRunner.doRun(TestRunner.java:116)
> at junit.textui.TestRunner.doRun(TestRunner.java:109)
> at
> org.apache.jdo.tck.util.BatchTestRunner.start(BatchTestRunner.java:120)
> at org.apache.jdo.tck.util.BatchTestRunner.main(BatchTestRunner.java:95)
> .
> at org.jpox.store.query.JDOQLQuery.performExecute(JDOQLQuery.java:747)
> at org.jpox.store.query.Query.executeWithMap(Query.java:966)
> at org.jpox.store.query.Query.executeWithArray(Query.java:939)
> at org.jpox.store.query.Query.execute(Query.java:862)
> at org.apache.jdo.tck.query.QueryTest.execute(QueryTest.java:1151)
> at org.apache.jdo.tck.query.QueryTest.execute(QueryTest.java:1029)
> at
> org.apache.jdo.tck.query.QueryTest.executeAPIQuery(QueryTest.java:966)
> at
> org.apache.jdo.tck.query.QueryTest.executeAPIQuery(QueryTest.java:946)
> at org.apache.jdo.tck.query.result.Having.testPositive(Having.java:110)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
> at java.lang.reflect.Method.invoke(Method.java:324)
> at junit.framework.TestCase.runTest(TestCase.java:154)
> at org.apache.jdo.tck.JDO_Test.runBare(JDO_Test.java:204)
> at junit.framework.TestResult$1.protect(TestResult.java:106)
> at junit.framework.TestResult.runProtected(TestResult.java:124)
> at junit.framework.TestResult.run(TestResult.java:109)
> at junit.framework.TestCase.run(TestCase.java:118)
> at junit.framework.TestSuite.runTest(TestSuite.java:208)
> at junit.framework.TestSuite.run(TestSuite.java:203)
> at junit.framework.TestSuite.runTest(TestSuite.java:208)
> at junit.framework.TestSuite.run(TestSuite.java:203)
> at junit.textui.TestRunner.doRun(TestRunner.java:116)
> at junit.textui.TestRunner.doRun(TestRunner.java:109)
> at
> org.apache.jdo.tck.util.BatchTestRunner.start(BatchTestRunner.java:120)
> at org.apache.jdo.tck.util.BatchTestRunner.main(BatchTestRunner.java:95)
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
