You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by jl...@apache.org on 2020/09/04 17:41:47 UTC

[incubator-pinot] branch remove-host-from-swagger created (now 6fdae11)

This is an automated email from the ASF dual-hosted git repository.

jlli pushed a change to branch remove-host-from-swagger
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


      at 6fdae11  Remove host from swagger setting

This branch includes the following new commits:

     new 6fdae11  Remove host from swagger setting

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[incubator-pinot] 01/01: Remove host from swagger setting

Posted by jl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jlli pushed a commit to branch remove-host-from-swagger
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit 6fdae114f53b86c3bf19363ef3886d6720330446
Author: Jack Li(Analytics Engineering) <jl...@jlli-mn1.linkedin.biz>
AuthorDate: Fri Sep 4 10:41:26 2020 -0700

    Remove host from swagger setting
---
 .../apache/pinot/controller/api/ControllerAdminApiApplication.java | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java b/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java
index ed77afe..5c2c8e3 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java
@@ -153,13 +153,6 @@ public class ControllerAdminApiApplication extends ResourceConfig {
     beanConfig.setResourcePackage(RESOURCE_PACKAGE);
     beanConfig.setScan(true);
 
-    Collection<NetworkListener> listeners = httpServer.getListeners();
-    if (listeners.size() > 0) {
-      // fetch the port from the first listener which uses http
-      int port = listeners.iterator().next().getPort();
-      beanConfig.setHost("localhost:" + port);
-    }
-
     ClassLoader loader = this.getClass().getClassLoader();
     CLStaticHttpHandler apiStaticHttpHandler = new CLStaticHttpHandler(loader, "/api/");
     // map both /api and /help to swagger docs. /api because it looks nice. /help for backward compatibility


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org