[GitHub] [ignite-3] zstan commented on a change in pull request #672: IGNITE-16266 Add unique id for indexes

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

zstan commented on a change in pull request #672:
URL: https://github.com/apache/ignite-3/pull/672#discussion_r840430680



##########
File path: modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/externalize/RelInputEx.java
##########
@@ -38,8 +40,17 @@
     /**
      * Returns table by its id.
      *
-     * @param tag Tag under which id is serialised.
+     * @param tag Tag under which id is serialized.
      * @return A table with given id.
      */
     RelOptTable getTableById(String tag);
+
+    /**
+     * Check that appropriate index is available.
+     *
+     * @param tag Tag represents id serialization.
+     * @param idxName Index name.

Review comment:
       What kind of information will obtain user in case of error in such a case ? I think that UUID is also not for user eyes but it`s discussable.




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