You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by am...@apache.org on 2021/08/02 12:06:04 UTC

[ignite-3] 03/05: Minor. add more todos.

This is an automated email from the ASF dual-hosted git repository.

amashenkov pushed a commit to branch ignite-15212
in repository https://gitbox.apache.org/repos/asf/ignite-3.git

commit 43041d5a0f58a9e91cc319e64b62fea6b6b29bcf
Author: Andrew Mashenkov <an...@gmail.com>
AuthorDate: Thu Jul 29 18:41:54 2021 +0300

    Minor. add more todos.
---
 .../java/org/apache/ignite/query/sql/IgniteSql.java     | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/modules/api/src/main/java/org/apache/ignite/query/sql/IgniteSql.java b/modules/api/src/main/java/org/apache/ignite/query/sql/IgniteSql.java
index a44ab14..395100e 100644
--- a/modules/api/src/main/java/org/apache/ignite/query/sql/IgniteSql.java
+++ b/modules/api/src/main/java/org/apache/ignite/query/sql/IgniteSql.java
@@ -67,12 +67,6 @@ public interface IgniteSql {
 
     ///////////////// Query monitoring and management.
     /**
-     * Returns SQL views.
-     */
-    Object views(); //TODO: To be described.
-    //TODO: Add view for runnin queries.
-
-    /**
      * Return info of the queries. //TODO: "running on the node locally"? or "started on the node"? or both?
      *
      * @return Running queries infos.
@@ -86,6 +80,15 @@ public interface IgniteSql {
      */
     void killQuery(UUID queryID);
 
+    /**
+     * Returns SQL views.
+     */
+    Object views(); //TODO: TBD.
+
+    //TODO: View for running queries?
+    //TODO: View for memory Management?
+    //TODO: View for QueryMetrics?
+    //TODO: Any other SystemViews from Igntie 2.0?
 
     ////////////// Statistics management.
     Collection<Object> tableStatistics(String table); //TODO: Local or global? Ready or in-progress? TBD.
@@ -105,5 +108,7 @@ public interface IgniteSql {
 
     //TODO: Do we expect any SQL DDL pragrammatical API here? Why we have tables().create(), but not here?
     //TODO: alterTable()?
+
+    //TODO: Custom function registration. Do we need a view and unregister functionality?
 }