ibessonov commented on code in PR #5047:
URL: https://github.com/apache/ignite-3/pull/5047#discussion_r1914861666
##########
modules/rest/src/main/java/org/apache/ignite/internal/rest/cluster/ClusterManagementController.java:
##########
@@ -86,7 +88,14 @@ public CompletableFuture<Void> init(@Body InitCommand
initCommand) {
});
}
- private static ClusterState
mapClusterState(org.apache.ignite.internal.cluster.management.ClusterState
clusterState) {
+ private static ClusterState mapClusterState(@Nullable
org.apache.ignite.internal.cluster.management.ClusterState clusterState) {
+ if (clusterState == null) {
+ throw new IgniteException(
+ INTERNAL_ERR,
Review Comment:
Ideally, it should not be an internal error. Maybe we should introduce a new
exception and code?
--
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]
