This is an automated email from the ASF dual-hosted git repository.
kfaraz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new b27d281b11 Remove unused param in MetadataResource (#14747)
b27d281b11 is described below
commit b27d281b113a32aa1c5115f98c46a586f0df8138
Author: Kashif Faraz <[email protected]>
AuthorDate: Thu Aug 3 19:18:01 2023 +0530
Remove unused param in MetadataResource (#14747)
---
.../main/java/org/apache/druid/server/http/MetadataResource.java | 7 ++-----
.../java/org/apache/druid/server/http/MetadataResourceTest.java | 4 +---
2 files changed, 3 insertions(+), 8 deletions(-)
diff --git
a/server/src/main/java/org/apache/druid/server/http/MetadataResource.java
b/server/src/main/java/org/apache/druid/server/http/MetadataResource.java
index 78d799b451..6cdf9c546a 100644
--- a/server/src/main/java/org/apache/druid/server/http/MetadataResource.java
+++ b/server/src/main/java/org/apache/druid/server/http/MetadataResource.java
@@ -19,7 +19,6 @@
package org.apache.druid.server.http;
-import com.fasterxml.jackson.databind.ObjectMapper;
import com.google.common.base.Function;
import com.google.common.collect.Collections2;
import com.google.common.collect.Iterables;
@@ -27,7 +26,6 @@ import com.google.inject.Inject;
import com.sun.jersey.spi.container.ResourceFilters;
import org.apache.druid.client.DataSourcesSnapshot;
import org.apache.druid.client.ImmutableDruidDataSource;
-import org.apache.druid.guice.annotations.Json;
import org.apache.druid.indexing.overlord.IndexerMetadataStorageCoordinator;
import org.apache.druid.indexing.overlord.Segments;
import org.apache.druid.metadata.SegmentsMetadataManager;
@@ -77,8 +75,7 @@ public class MetadataResource
SegmentsMetadataManager segmentsMetadataManager,
IndexerMetadataStorageCoordinator metadataStorageCoordinator,
AuthorizerMapper authorizerMapper,
- DruidCoordinator coordinator,
- @Json ObjectMapper jsonMapper
+ DruidCoordinator coordinator
)
{
this.segmentsMetadataManager = segmentsMetadataManager;
@@ -282,7 +279,7 @@ public class MetadataResource
@Path("/datasources/{dataSourceName}/segments/{segmentId}")
@Produces(MediaType.APPLICATION_JSON)
@ResourceFilters(DatasourceResourceFilter.class)
- public Response isSegmentUsed(
+ public Response getUsedSegment(
@PathParam("dataSourceName") String dataSourceName,
@PathParam("segmentId") String segmentId
)
diff --git
a/server/src/test/java/org/apache/druid/server/http/MetadataResourceTest.java
b/server/src/test/java/org/apache/druid/server/http/MetadataResourceTest.java
index f030345179..e59e149d8c 100644
---
a/server/src/test/java/org/apache/druid/server/http/MetadataResourceTest.java
+++
b/server/src/test/java/org/apache/druid/server/http/MetadataResourceTest.java
@@ -19,7 +19,6 @@
package org.apache.druid.server.http;
-import com.fasterxml.jackson.databind.ObjectMapper;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
@@ -91,8 +90,7 @@ public class MetadataResourceTest
segmentsMetadataManager,
Mockito.mock(IndexerMetadataStorageCoordinator.class),
AuthTestUtils.TEST_AUTHORIZER_MAPPER,
- coordinator,
- new ObjectMapper()
+ coordinator
);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
