navina commented on PR #12222: URL: https://github.com/apache/pinot/pull/12222#issuecomment-1883590302
> Hey @walterddr @navina can you help review this PR? Should we remove boolean check altogether or add String / Numeric as additional checks? i think it will be better to add additional string/numeric checks in TableConfigUtils. Do you think that is possible? Apologies for the delay in response @tibrewalpratik17 . I missed the notification. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
