Skip to content

Commit f982830

Browse files
kotwanikunalashking94
authored andcommitted
Remove unused object and import from TransportClusterAllocationExplainAction (opensearch-project#4639)
Signed-off-by: Kunal Kotwani <[email protected]> Signed-off-by: Kunal Kotwani <[email protected]>
1 parent 0a7bee1 commit f982830

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
5757

5858
### Removed
5959
- Remove deprecated code to add node name into log pattern of log4j property file ([#4568](https://github.com/opensearch-project/OpenSearch/pull/4568))
60+
- Unused object and import within TransportClusterAllocationExplainAction ([#4639](https://github.com/opensearch-project/OpenSearch/pull/4639))
6061

6162
### Fixed
6263
- `opensearch-service.bat start` and `opensearch-service.bat manager` failing to run ([#4289](https://github.com/opensearch-project/OpenSearch/pull/4289))
@@ -112,4 +113,4 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
112113

113114

114115
[Unreleased]: https://github.com/opensearch-project/OpenSearch/compare/2.2.0...HEAD
115-
[2.x]: https://github.com/opensearch-project/OpenSearch/compare/2.2.0...2.x
116+
[2.x]: https://github.com/opensearch-project/OpenSearch/compare/2.2.0...2.x

server/src/main/java/org/opensearch/action/admin/cluster/allocation/TransportClusterAllocationExplainAction.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@
5050
import org.opensearch.cluster.routing.allocation.RoutingAllocation;
5151
import org.opensearch.cluster.routing.allocation.RoutingAllocation.DebugMode;
5252
import org.opensearch.cluster.routing.allocation.ShardAllocationDecision;
53-
import org.opensearch.cluster.routing.allocation.allocator.ShardsAllocator;
5453
import org.opensearch.cluster.routing.allocation.decider.AllocationDeciders;
5554
import org.opensearch.cluster.service.ClusterService;
5655
import org.opensearch.common.inject.Inject;
@@ -77,7 +76,6 @@ public class TransportClusterAllocationExplainAction extends TransportClusterMan
7776
private final ClusterInfoService clusterInfoService;
7877
private final SnapshotsInfoService snapshotsInfoService;
7978
private final AllocationDeciders allocationDeciders;
80-
private final ShardsAllocator shardAllocator;
8179
private final AllocationService allocationService;
8280

8381
@Inject
@@ -90,7 +88,6 @@ public TransportClusterAllocationExplainAction(
9088
ClusterInfoService clusterInfoService,
9189
SnapshotsInfoService snapshotsInfoService,
9290
AllocationDeciders allocationDeciders,
93-
ShardsAllocator shardAllocator,
9491
AllocationService allocationService
9592
) {
9693
super(
@@ -105,7 +102,6 @@ public TransportClusterAllocationExplainAction(
105102
this.clusterInfoService = clusterInfoService;
106103
this.snapshotsInfoService = snapshotsInfoService;
107104
this.allocationDeciders = allocationDeciders;
108-
this.shardAllocator = shardAllocator;
109105
this.allocationService = allocationService;
110106
}
111107

0 commit comments

Comments
 (0)