[incubator-pinot] branch master updated: Include exception message in the ControllerApplicationException for /validate (#6312)

[xiangfu]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 9ce5d78  Include exception message in the 
ControllerApplicationException for /validate (#6312)
9ce5d78 is described below

commit 9ce5d78e1595d7889e903066d84684b284eb676b
Author: Neha Pawar <neha.pawa...@gmail.com>
AuthorDate: Thu Dec 3 14:07:35 2020 -0800

    Include exception message in the ControllerApplicationException for 
/validate (#6312)
---
 .../pinot/controller/api/resources/PinotTableRestletResource.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java
 
b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java
index 116ad35..50940ea 100644
--- 
a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java
+++ 
b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java
@@ -390,7 +390,7 @@ public class PinotTableRestletResource {
       }
       return tableConfigValidateStr.toString();
     } catch (Exception e) {
-      String msg = String.format("Invalid table config: %s", 
tableConfig.getTableName());
+      String msg = String.format("Invalid table config: %s. %s", 
tableConfig.getTableName(), e.getMessage());
       throw new ControllerApplicationException(LOGGER, msg, 
Response.Status.BAD_REQUEST, e);
     }
   }


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