From b7da601d88e296f0f7ad5b225190c4be1511b484 Mon Sep 17 00:00:00 2001
From: Lakshya Taragi <157457166+ltaragi@users.noreply.github.com>
Date: Mon, 16 Sep 2024 14:58:07 +0530
Subject: [PATCH] Remove TooManyShardsInSnapshotsStatusException (#15877)

Signed-off-by: Lakshya Taragi <lakshya.taragi@gmail.com>
---
 .../opensearch/OpenSearchServerException.java |  8 ---
 ...oManyShardsInSnapshotsStatusException.java | 69 -------------------
 .../ExceptionSerializationTests.java          |  2 -
 3 files changed, 79 deletions(-)
 delete mode 100644 server/src/main/java/org/opensearch/snapshots/TooManyShardsInSnapshotsStatusException.java

diff --git a/server/src/main/java/org/opensearch/OpenSearchServerException.java b/server/src/main/java/org/opensearch/OpenSearchServerException.java
index a1fc61834f69b..b0ab62259d5c0 100644
--- a/server/src/main/java/org/opensearch/OpenSearchServerException.java
+++ b/server/src/main/java/org/opensearch/OpenSearchServerException.java
@@ -1210,14 +1210,6 @@ public static void registerExceptions() {
                 V_2_17_0
             )
         );
-        registerExceptionHandle(
-            new OpenSearchExceptionHandle(
-                org.opensearch.snapshots.TooManyShardsInSnapshotsStatusException.class,
-                org.opensearch.snapshots.TooManyShardsInSnapshotsStatusException::new,
-                175,
-                V_2_17_0
-            )
-        );
         registerExceptionHandle(
             new OpenSearchExceptionHandle(
                 org.opensearch.cluster.block.IndexCreateBlockException.class,
diff --git a/server/src/main/java/org/opensearch/snapshots/TooManyShardsInSnapshotsStatusException.java b/server/src/main/java/org/opensearch/snapshots/TooManyShardsInSnapshotsStatusException.java
deleted file mode 100644
index 1689b3e4941ec..0000000000000
--- a/server/src/main/java/org/opensearch/snapshots/TooManyShardsInSnapshotsStatusException.java
+++ /dev/null
@@ -1,69 +0,0 @@
-/*
- * SPDX-License-Identifier: Apache-2.0
- *
- * The OpenSearch Contributors require contributions made to
- * this file be licensed under the Apache-2.0 license or a
- * compatible open source license.
- */
-
-/*
- * Licensed to Elasticsearch under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-/*
- * Modifications Copyright OpenSearch Contributors. See
- * GitHub history for details.
- */
-
-package org.opensearch.snapshots;
-
-import org.opensearch.core.common.io.stream.StreamInput;
-import org.opensearch.core.rest.RestStatus;
-
-import java.io.IOException;
-
-/**
- * Thrown if the number of shards across the requested resources (snapshot(s) or the index/indices of a particular snapshot)
- * breaches the limit of snapshot.max_shards_allowed_in_status_api cluster setting
- *
- * @opensearch.internal
- */
-public class TooManyShardsInSnapshotsStatusException extends SnapshotException {
-
-    public TooManyShardsInSnapshotsStatusException(
-        final String repositoryName,
-        final SnapshotId snapshotId,
-        final String message,
-        final Throwable cause
-    ) {
-        super(repositoryName, snapshotId, message, cause);
-    }
-
-    public TooManyShardsInSnapshotsStatusException(final String repositoryName, final String message, String... snapshotName) {
-        super(repositoryName, String.join(", ", snapshotName), message);
-    }
-
-    public TooManyShardsInSnapshotsStatusException(StreamInput in) throws IOException {
-        super(in);
-    }
-
-    @Override
-    public RestStatus status() {
-        return RestStatus.REQUEST_ENTITY_TOO_LARGE;
-    }
-}
diff --git a/server/src/test/java/org/opensearch/ExceptionSerializationTests.java b/server/src/test/java/org/opensearch/ExceptionSerializationTests.java
index eff312a36dbc6..2e4a2d7bdd59c 100644
--- a/server/src/test/java/org/opensearch/ExceptionSerializationTests.java
+++ b/server/src/test/java/org/opensearch/ExceptionSerializationTests.java
@@ -119,7 +119,6 @@
 import org.opensearch.snapshots.SnapshotId;
 import org.opensearch.snapshots.SnapshotInProgressException;
 import org.opensearch.snapshots.SnapshotInUseDeletionException;
-import org.opensearch.snapshots.TooManyShardsInSnapshotsStatusException;
 import org.opensearch.test.OpenSearchTestCase;
 import org.opensearch.test.VersionUtils;
 import org.opensearch.transport.ActionNotFoundTransportException;
@@ -899,7 +898,6 @@ public void testIds() {
         ids.put(172, ViewNotFoundException.class);
         ids.put(173, ViewAlreadyExistsException.class);
         ids.put(174, InvalidIndexContextException.class);
-        ids.put(175, TooManyShardsInSnapshotsStatusException.class);
         ids.put(10001, IndexCreateBlockException.class);
 
         Map<Class<? extends OpenSearchException>, Integer> reverse = new HashMap<>();