You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/07/13 12:24:17 UTC

[GitHub] [inlong] leosanqing commented on a diff in pull request #5011: [INLONG-5007][Manager] Client supports getting sink details by id

leosanqing commented on code in PR #5011:
URL: https://github.com/apache/inlong/pull/5011#discussion_r920014230


##########
inlong-manager/manager-client/src/main/java/org/apache/inlong/manager/client/api/InlongStream.java:
##########
@@ -51,6 +51,16 @@ public interface InlongStream {
      */
     Map<String, StreamSink> getSinks();
 
+    /**
+     * Get detail info of data sink by id.
+     */
+    StreamSink getSinkInfoById(Integer sinkId);
+
+    /**
+     * Get detail info of data sink by name.
+     */
+    StreamSink getSinkInfoByName(String sinkName);

Review Comment:
   This is obtained from the SinkMap in StreamImpl, and it is not queried from the database. The duplicate name check has been performed when the map is generated.



-- 
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-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org