walterddr commented on code in PR #9006: URL: https://github.com/apache/pinot/pull/9006#discussion_r918044682
########## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentUploadDownloadRestletResource.java: ########## @@ -652,6 +652,29 @@ public Response endReplaceSegments( } } + @GET + @Path("segments/{tableName}/listSegmentLineage") + @Authenticate(AccessType.READ) + @Produces(MediaType.APPLICATION_JSON) + @ApiOperation(value = "List segment lineage", notes = "List segment lineage") + public Response listSegmentLineage( Review Comment: oh. actually my mistake. I think the first thing I suggest is to move the API from `PinotSegmentUploadDownloadRestletResource` to `PinotSegmentRestletResource` where all the other segment metadata APIs are located. then it should be easy to add a test in `PinotSegmentRestletResourceTest` -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org