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/10/13 23:37:39 UTC

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #6129: Issue #4854 Framework for adding compatibility tests

mcvsubbu commented on pull request #6129:
URL: https://github.com/apache/incubator-pinot/pull/6129#issuecomment-708066401


   we should remove all sleeps. They were added by the earlier committer. I will do that. 
   We also need to check the status of each compat-tester callout.
   Lastly, I think this whole thing needs to move into pinot-integration-tests, so that we can re-use all the code in there in the compat test operations. What do you say?


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