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


##########
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? Should we 
use concrete name of system schema instead of interesting by all of schemas?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to