LakshSingla commented on code in PR #14534:
URL: https://github.com/apache/druid/pull/14534#discussion_r1257963381


##########
processing/src/main/java/org/apache/druid/error/InvalidInput.java:
##########
@@ -35,7 +35,7 @@ public static DruidException exception(Throwable t, String 
msg, Object... args)
   private final String msg;
   private final Object[] args;
 
-  public InvalidInput(
+  protected InvalidInput(

Review Comment:
   Changed this because we had a couple of static InvalidInput exception 
creators that mimic this signature `exception(...)`, therefore any piece of 
code that intends to call this constructor can use the creator instead for the 
time being.
   
   Reverted back the change in my newer commit. 



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to