ibessonov commented on code in PR #1325:
URL: https://github.com/apache/ignite-3/pull/1325#discussion_r1032126134
##########
modules/storage-api/src/main/java/org/apache/ignite/internal/storage/engine/MvTableStorage.java:
##########
@@ -59,6 +60,10 @@ public interface MvTableStorage {
/**
* Destroys a partition and all associated indices.
*
+ * <p>This method will do nothing if there is no partition by ID, when
trying to call methods to read or write (as well as all previous
+ * open cursors) for {@link MvPartitionStorage}, {@link HashIndexStorage}
and {@link SortedIndexStorage}, {@link StorageClosedException}
+ * will be thrown.
+ *
Review Comment:
The grammatical structure is very convoluted. Maybe something like
```
This method will ... if:
- blah
- foo
- bar
```
will be easier to read
--
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]
