You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/03/05 16:30:57 UTC

[GitHub] [incubator-pinot] xiaohui-sun commented on issue #5113: [TE] added a couple of validations to check if detection is subscribed & valid; cleaned up other validations

xiaohui-sun commented on issue #5113: [TE] added a couple of validations to check if detection is subscribed & valid; cleaned up other validations
URL: https://github.com/apache/incubator-pinot/pull/5113#issuecomment-595321310
 
 
   Thanks for the clean up. I am thinking of refactoring the validation module to:
   1. Can expose multiple error messages together.
   2. Easier to add more validation rules.

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


With regards,
Apache Git Services

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