You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@isis.apache.org by mg...@apache.org on 2015/04/17 22:42:45 UTC

isis git commit: ISIS-1134 Consume the resultset in one more place

Repository: isis
Updated Branches:
  refs/heads/ISIS-1134-leaking-queryresults b126214a7 -> 57fb39590


ISIS-1134 Consume the resultset in one more place


Project: http://git-wip-us.apache.org/repos/asf/isis/repo
Commit: http://git-wip-us.apache.org/repos/asf/isis/commit/57fb3959
Tree: http://git-wip-us.apache.org/repos/asf/isis/tree/57fb3959
Diff: http://git-wip-us.apache.org/repos/asf/isis/diff/57fb3959

Branch: refs/heads/ISIS-1134-leaking-queryresults
Commit: 57fb39590168d5f701bc63b7b2e261fdbb3e73f6
Parents: b126214
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Fri Apr 17 23:42:15 2015 +0300
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Fri Apr 17 23:42:15 2015 +0300

----------------------------------------------------------------------
 .../queries/PersistenceQueryFindUsingApplibQueryProcessor.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/isis/blob/57fb3959/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/persistence/queries/PersistenceQueryFindUsingApplibQueryProcessor.java
----------------------------------------------------------------------
diff --git a/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/persistence/queries/PersistenceQueryFindUsingApplibQueryProcessor.java b/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/persistence/queries/PersistenceQueryFindUsingApplibQueryProcessor.java
index 9744392..f6361e1 100644
--- a/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/persistence/queries/PersistenceQueryFindUsingApplibQueryProcessor.java
+++ b/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/persistence/queries/PersistenceQueryFindUsingApplibQueryProcessor.java
@@ -116,7 +116,7 @@ public class PersistenceQueryFindUsingApplibQueryProcessor extends PersistenceQu
         try {
             final List<?> results = (List<?>) jdoQuery.executeWithMap(argumentsByParameterName);
             if (cardinality == QueryCardinality.MULTIPLE) {
-                return results;
+                return Lists.newArrayList(results);
             }
             return results.isEmpty() ? Collections.emptyList() : Lists.newArrayList(results.subList(0, 1));
         } finally {