[GitHub] [pulsar] freeznet commented on a change in pull request #9246: [fix #8089] expose PulsarAdmin client through Function Context

[GitBox <gi...@apache.org>]

freeznet commented on a change in pull request #9246:
URL: https://github.com/apache/pulsar/pull/9246#discussion_r561449955



##########
File path: pulsar-functions/api-java/src/main/java/org/apache/pulsar/functions/api/Context.java
##########
@@ -262,6 +264,21 @@
      */
     String getSecret(String secretName);
 
+    /**
+     * Get the pulsar admin client.
+     *
+     * @return The instance of pulsar admin client
+     */
+    PulsarAdmin getPulsarAdmin();
+
+    /**
+     * Get the pulsar admin client by cluster name.

Review comment:
       yes, just as sijie said, `getPulsarAdmin()` will just get the `default` pulsar admin client, the other one will let user get pulsar admin client if `externalPulsarClusters` is set.




----------------------------------------------------------------
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.

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