[GitHub] [ignite] ivandasch commented on a change in pull request #9821: IGNITE-16151 Introduce query planner timeout.

[GitBox <gi...@apache.org>]

ivandasch commented on a change in pull request #9821:
URL: https://github.com/apache/ignite/pull/9821#discussion_r808881246



##########
File path: modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/CalciteQueryProcessor.java
##########
@@ -88,8 +89,23 @@
 import org.apache.ignite.internal.processors.query.calcite.util.Service;
 import org.jetbrains.annotations.Nullable;
 
+import static org.apache.ignite.IgniteSystemProperties.getLong;
+
 /** */
 public class CalciteQueryProcessor extends GridProcessorAdapter implements QueryEngine {
+    /**
+     * Default planner timeout, in ms.
+     */
+    private static final long DFLT_IGNITE_EXPERIMENTAL_SQL_ENGINE_PLANNER_TIMEOUT = 15000;
+
+    /**
+     * Determines whether to use the experimental sql, calcite based, engine.
+     */
+    @SystemProperty(value = "Timeout of experimental sql engine planner timeout, in ms", type = Long.class,

Review comment:
       Oh, great catch, thank you




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org