You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2019/01/25 10:26:41 UTC

[GitHub] AMashenkov commented on a change in pull request #5875: IGNITE-10305

AMashenkov commented on a change in pull request #5875: IGNITE-10305
URL: https://github.com/apache/ignite/pull/5875#discussion_r250935738
 
 

 ##########
 File path: modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/twostep/GridReduceQueryExecutor.java
 ##########
 @@ -1338,4 +1345,36 @@ private static long retryTimeout(long qryTimeout) {
 
         return IgniteSystemProperties.getLong(IGNITE_SQL_RETRY_TIMEOUT, DFLT_RETRY_TIMEOUT);
     }
+
+
+    /**
+     * Prepare map query based on original sql.
+     *
+     * @param qry Two step query.
+     * @param params Query parameters.
+     * @return Updated map query list with one map query.
+     */
+    private List<GridCacheSqlQuery> prepareMapQueryForSinglePartition(GridCacheTwoStepQuery qry, Object[] params) {
+        boolean hasSubQries = false;
+
+        for (GridCacheSqlQuery mapQry : qry.mapQueries())
+            hasSubQries |= mapQry.hasSubQueries();
 
 Review comment:
   Seems, there is no need to check all queries, but find only first which has sub-query.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services