You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by "tkalkirill (via GitHub)" <gi...@apache.org> on 2023/05/18 07:01:05 UTC

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2082: IGNITE-19509 Get rid of reading configuration when calling methods MvTableStorage#getOrCreate*Index

tkalkirill commented on code in PR #2082:
URL: https://github.com/apache/ignite-3/pull/2082#discussion_r1197475209


##########
modules/index/src/main/java/org/apache/ignite/internal/index/IndexBuilder.java:
##########
@@ -98,10 +99,10 @@ void stop() {
     /**
      * Initializes the build of the index.
      */
-    void startIndexBuild(TableIndexView tableIndexView, TableImpl table) {
+    void startIndexBuild(TableIndexView tableIndexView, TableImpl table, IndexDescriptor indexDescriptor) {

Review Comment:
   I think that in the near future this code will disappear, the building of indexes has changed a little in the https://github.com/apache/ignite-3/pull/2030



-- 
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