kfaraz commented on PR #16174:
URL: https://github.com/apache/druid/pull/16174#issuecomment-2014089434

   > What kind of validation and exception did you have in mind? Currently, the 
code doesn't perform any type checks for the columns.
   
   I should think that JDBI would throw some exception if we try to bind am 
invalid string which contains, say a substring like `'); DROP TABLE abc;`.
   We could try to verify that behaviour. By the way, this is optional and need 
not block this PR.


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