korlov42 commented on code in PR #2678:
URL: https://github.com/apache/ignite-3/pull/2678#discussion_r1358153630


##########
modules/catalog/src/main/java/org/apache/ignite/internal/catalog/CatalogManagerImpl.java:
##########
@@ -537,4 +556,88 @@ public List<UpdateEntry> get(Catalog catalog) {
             return bulkUpdateEntries;
         }
     }
+
+    private SystemView<?> createSystemViewsView() {
+        Iterable<SchemaAwareDescriptor<CatalogSystemViewDescriptor>> viewData 
= () -> {
+            int version = latestCatalogVersion();
+
+            Catalog catalog = catalog(version);
+
+            return catalog.schemas().stream()

Review Comment:
   > we supposed that we could have a few shcemas with a system views?
   from catalog's point of view, system view is just another object. like table 
or index. Thus, if anyone has managed to add system view to schema, it must be 
exposed by SYSTEM_VIEWS view 



-- 
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-unsubscr...@ignite.apache.org

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

Reply via email to