[GitHub] [incubator-pinot] amrishlal commented on pull request #6326: pinot-controller unit test suite.

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

amrishlal commented on pull request #6326:
URL: https://github.com/apache/incubator-pinot/pull/6326#issuecomment-744172897


   > While the PR indicates so, still would like to get a confirmation on the following:
   > 
   > * No regression in coverage, and developer productivity (in terms of being able to run/debug these tests individually).
   > * No existing tests were disabled.
   
   Confirming all three :-) These changes should not cause any change in overall test coverage. I verified that it is possible to run each test class individually and no test cases are disabled. Number of test cases that run before and after this change is the same (208).


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