timoninmaxim commented on code in PR #12378:
URL: https://github.com/apache/ignite/pull/12378#discussion_r2401421652
##########
modules/core/src/main/java/org/apache/ignite/internal/cache/query/index/IndexQueryProcessor.java:
##########
@@ -84,6 +84,11 @@ public <K, V> IndexQueryResult<K, V> queryLocal(
) throws IgniteCheckedException {
InlineIndexImpl idx = (InlineIndexImpl)findSortedIndex(cctx,
idxQryDesc);
+ if (idx.rebuildInProgress()) {
+ throw new IgniteCheckedException("Failed to run IndexQuery: index
" + idx.name() + " isn't completed yet."
Review Comment:
Let's make error message more standart:
`Failed to run IndexQuery due to index rebuild is in progress
[index=idx.indexDefinition().idxName(), query=idxQryDesc]
`
Usually if error message contains two or more params then we use notation
"err msg [param1=, param2=]".
`idx.name` - will return name of index, but not schema for which this index
is created (it could be a few indexes with the same name, but they are part of
different 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]
