korlov42 commented on code in PR #1005:
URL: https://github.com/apache/ignite-3/pull/1005#discussion_r955982257
##########
modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java:
##########
@@ -225,13 +256,32 @@ public CompletableFuture<Index<?>> createIndexAsync(
*
* @param schemaName A name of the schema the index belong to.
* @param indexName A name of the index to drop.
+ * @param failIfNotExist Flag, which force failure, when {@code trues} if
index doen't not exists.
+ * @return {@code True} if index was removed, {@code false} otherwise.
+ * @throws IndexNotFoundException If index doesn't exist and {@code
failIfNotExist} param was {@code true}.
+ */
+ public boolean dropIndex(
Review Comment:
This is actually a technical debt that DdlHandler uses async api. Well, I'm
ok to leave this as is. But in the future we need to not forget to clean this
after DdlHandler be reworked in order to use async api
--
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]
