wirybeaver commented on code in PR #17521:
URL: https://github.com/apache/pinot/pull/17521#discussion_r2970738573
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -415,6 +434,18 @@ static void tweakRealtimeTableConfig(ObjectNode
realtimeTableConfigNode, CopyTab
}
}
+ @GET
+ @Path("/tables/copyStatus/{jobId}")
+ @Authorize(targetType = TargetType.CLUSTER, action =
Actions.Cluster.GET_TABLE_COPY_STATUS)
+ @Produces(MediaType.APPLICATION_JSON)
+ @ApiOperation(value = "Get status for a submitted table replication job",
+ notes = "Get status for a submitted table replication job")
+ public JsonNode getForceCommitJobStatus(
Review Comment:
Done in e5e31e001c — renamed to `getTableCopyStatus` to match the API path.
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -415,6 +434,18 @@ static void tweakRealtimeTableConfig(ObjectNode
realtimeTableConfigNode, CopyTab
}
}
+ @GET
+ @Path("/tables/copyStatus/{jobId}")
+ @Authorize(targetType = TargetType.CLUSTER, action =
Actions.Cluster.GET_TABLE_COPY_STATUS)
+ @Produces(MediaType.APPLICATION_JSON)
+ @ApiOperation(value = "Get status for a submitted table replication job",
+ notes = "Get status for a submitted table replication job")
+ public JsonNode getForceCommitJobStatus(
+ @ApiParam(value = "job id", required = true) @PathParam("jobId") String
id) {
+ return JsonUtils.objectToJsonNode(
+ _pinotHelixResourceManager.getControllerJobZKMetadata(id,
ControllerJobTypes.TABLE_REPLICATION));
Review Comment:
Done in e5e31e001c — now returns 404 when the job ID is not found in ZK.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
