senlizishi commented on issue #7591:
URL: https://github.com/apache/gravitino/issues/7591#issuecomment-3044039148

   Hi @justinmclean I looked at the code and was a bit confused. 
   
   Firstly, the comment for isIfExists is` /* * @ return If true, silence the 
error if index does not exist during drop. */`This is different from its own 
meaning and looks more like a switch than whether it exists or not. 
   
   
   Next, If anyMatch has an Index., throw Index does not exist. 
    Is it a logical error?  Could you give me some guidance?  Thx
   ```
   if (Arrays.stream(lazyLoadTable.index())
             .anyMatch(index -> index.name().equals(deleteIndex.getName()))) {
           throw new IllegalArgumentException("Index does not exist");
         }
   ```
   


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