[pinot] branch master updated: Avoid swagger loading too much types and showing noisy log (#9144)

[jackie]

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

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new dad2586d5d Avoid swagger loading too much types and showing noisy log 
(#9144)
dad2586d5d is described below

commit dad2586d5d313e7d828f183827a9e5379abe64f9
Author: Xiaoman Dong <xiao...@startree.ai>
AuthorDate: Tue Aug 2 11:27:41 2022 -0700

    Avoid swagger loading too much types and showing noisy log (#9144)
---
 .../java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java  | 1 +
 .../org/apache/pinot/controller/api/ControllerAdminApiApplication.java  | 1 +
 .../main/java/org/apache/pinot/minion/MinionAdminApiApplication.java    | 1 +
 .../java/org/apache/pinot/server/starter/helix/AdminApiApplication.java | 1 +
 .../pinot/tools/service/PinotServiceManagerAdminApiApplication.java     | 2 +-
 5 files changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java
 
b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java
index cfc5d55928..fcf151fab6 100644
--- 
a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java
+++ 
b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java
@@ -91,6 +91,7 @@ public class BrokerAdminApiApplication extends ResourceConfig 
{
     beanConfig.setDescription("APIs for accessing Pinot broker information");
     beanConfig.setContact("https://github.com/apache/pinot";);
     beanConfig.setVersion("1.0");
+    beanConfig.setExpandSuperTypes(false);
     if (_useHttps) {
       beanConfig.setSchemes(new String[]{CommonConstants.HTTPS_PROTOCOL});
     } else {
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 fa95790ad9..f7ef335ee1 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
@@ -111,6 +111,7 @@ public class ControllerAdminApiApplication extends 
ResourceConfig {
     beanConfig.setDescription("APIs for accessing Pinot Controller 
information");
     beanConfig.setContact("https://github.com/apache/pinot";);
     beanConfig.setVersion("1.0");
+    beanConfig.setExpandSuperTypes(false);
     if (_useHttps) {
       beanConfig.setSchemes(new String[]{CommonConstants.HTTPS_PROTOCOL});
     } else {
diff --git 
a/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java
 
b/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java
index 71f1ee6320..4616f06432 100644
--- 
a/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java
+++ 
b/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java
@@ -85,6 +85,7 @@ public class MinionAdminApiApplication extends ResourceConfig 
{
     beanConfig.setDescription("APIs for accessing Pinot Minion information");
     beanConfig.setContact("https://github.com/apache/pinot";);
     beanConfig.setVersion("1.0");
+    beanConfig.setExpandSuperTypes(false);
     if (_useHttps) {
       beanConfig.setSchemes(new String[]{CommonConstants.HTTPS_PROTOCOL});
     } else {
diff --git 
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java
 
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java
index edcd944c0c..aa645005a3 100644
--- 
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java
+++ 
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java
@@ -115,6 +115,7 @@ public class AdminApiApplication extends ResourceConfig {
     beanConfig.setDescription("APIs for accessing Pinot server information");
     beanConfig.setContact("https://github.com/apache/pinot";);
     beanConfig.setVersion("1.0");
+    beanConfig.setExpandSuperTypes(false);
     if 
(Boolean.parseBoolean(pinotConfiguration.getProperty(CommonConstants.Server.CONFIG_OF_SWAGGER_USE_HTTPS)))
 {
       beanConfig.setSchemes(new String[]{CommonConstants.HTTPS_PROTOCOL});
     } else {
diff --git 
a/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java
 
b/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java
index c765187754..20f5392401 100644
--- 
a/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java
+++ 
b/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java
@@ -73,7 +73,7 @@ public class PinotServiceManagerAdminApiApplication extends 
ResourceConfig {
     beanConfig.setBasePath(_baseUri.getPath());
     beanConfig.setResourcePackage(RESOURCE_PACKAGE);
     beanConfig.setScan(true);
-
+    beanConfig.setExpandSuperTypes(false);
     HttpHandler httpHandler =
         new 
CLStaticHttpHandler(PinotServiceManagerAdminApiApplication.class.getClassLoader(),
 "/api/");
     // map both /api and /help to swagger docs. /api because it looks nice. 
/help for backward compatibility


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