[GitHub] [incubator-pinot] npawar commented on pull request #5706: DateTimeFieldSpec validation in schema validate

[GitBox <gi...@apache.org>]

npawar commented on pull request #5706:
URL: https://github.com/apache/incubator-pinot/pull/5706#issuecomment-658900020


   > Recommend using exception to pass the failure message instead of passing in a logger for both simplicity and better control on the logging level
   
   The Schema validation code already was using a logger, and I was just trying to work around that. I'm not sure why it was done this way in Schema in the first place. I can change it to exception based throughout the Schema validation code path


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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