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

pratik 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 453121d787 improve error for schema name validation (#14517)
453121d787 is described below

commit 453121d78752bd9a2a50895bf2c6f771388335f2
Author: rohit <[email protected]>
AuthorDate: Fri Nov 22 12:45:10 2024 +0530

    improve error for schema name validation (#14517)
---
 .../pinot/controller/api/resources/PinotSchemaRestletResource.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java
 
b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java
index 8c85f6d355..d0a087c1d8 100644
--- 
a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java
+++ 
b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java
@@ -393,7 +393,8 @@ public class PinotSchemaRestletResource {
           "Invalid schema. Reason: 'schemaName' should not be null or empty", 
Response.Status.BAD_REQUEST);
     }
     if 
(schema.getSchemaName().contains(TableConfig.TABLE_NAME_FORBIDDEN_SUBSTRING)) {
-      throw new IllegalStateException("'schemaName' cannot contain double 
underscore ('__')");
+      throw new ControllerApplicationException(LOGGER, "'schemaName' cannot 
contain double underscore ('__')",
+          Response.Status.BAD_REQUEST);
     }
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to