Pawel Veselov created OPENJPA-2736:
--------------------------------------
Summary: Can not set sub-query parameters
Key: OPENJPA-2736
URL: https://issues.apache.org/jira/browse/OPENJPA-2736
Project: OpenJPA
Issue Type: Bug
Components: jpa
Affects Versions: 2.4.2
Reporter: Pawel Veselov
It seems that in some cases, the parameters can't be set for sub-queries. I've
seen this exception:
{noformat}
java.lang.IllegalArgumentException: Parameter named "device_group_access_hash"
is not declared in query "SELECT SUM(1) AS total_count, MIN(e.when) AS
min_time, MAX(e.when) AS max_time, e.vehicle.vin AS key_vin, e.level AS
key_level, e.moduleID AS key_json_moduleID FROM E_ComponentLog e INNER JOIN
e.vehicle ? WHERE (((e.vehicle.tenancy = :tenancy AND e.type = :_type) AND
e.when BETWEEN :minTime AND :maxTime) AND e.vehicle IN (SELECT DISTINCT * FROM
E_SotaDevice e INNER JOIN e.groups ? WHERE e.groups.group IN (SELECT
e.groupName FROM E_SotaDeviceGroupBlock e WHERE (e.strHash =
:device_group_access_hash AND e.tenancy = e.tenancy)))) GROUP BY e.vehicle.vin,
e.level, e.moduleID". Declared parameter keys are
"[ParameterExpression<Timestamp>('maxTime'),
ParameterExpression<String>('_type'), ParameterExpression<String>('tenancy'),
ParameterExpression<Timestamp>('minTime')]".
at
org.apache.openjpa.persistence.AbstractQuery.getParameter(AbstractQuery.java:385)
at
org.apache.openjpa.persistence.AbstractQuery.setParameter(AbstractQuery.java:586)
at
org.apache.openjpa.persistence.AbstractQuery.setParameter(AbstractQuery.java:47)
at java.util.HashMap.forEach(HashMap.java:1289)
at com.excelfore.util.Q$COMPONENT_LOG_GROUP.getGroupList(Q.java:2384)
{noformat}
The parameter can clearly be seen in the query. I don't really know how to
localize it into a test case. This only happens when
EntityManagerImpl.createQuery() ends up calling declareParameter(). It looks
like there are two ways parameters are processed, and for simpler queries, this
doesn't happen.
When this happens, the parameters are gathered using parameter visitors. But
subquery did not delegate its parameters to the visitor, so they were omitted.
I've fixed this by:
{noformat}
Index:
openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/SubqueryImpl.java
===================================================================
---
openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/SubqueryImpl.java
(revision 1828187)
+++
openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/SubqueryImpl.java
(working copy)
@@ -365,4 +361,18 @@
public StringBuilder asVariable(AliasContext q) {
return asValue(q);
}
+
+ @Override
+ public void acceptVisit(CriteriaExpressionVisitor visitor) {
+ // $TODO: this should be all expressions, not just parameters,
+ // but I don't know exactly how I would get those, plus this is so far
+ // only used for gathering parameters, so...
+ Set<ParameterExpression<?>> pSet = _delegate.getParameters();
+ Expression<?>[] expressions = new Expression<?>[pSet.size()];
+ int i = 0;
+ for (ParameterExpression pe : pSet) {
+ expressions[i++] = pe;
+ }
+ Expressions.acceptVisit(visitor, this, expressions);
+ }
{noformat}
Note, that because I have other fixes in this file due to OPENJPA-2733, so the
diff will need to be manually applied, and imports adjusted.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
