[GitHub] [druid] xvrl commented on issue #12399: Test Cases are using the print instead of Assert in testing

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

xvrl commented on issue #12399:
URL: https://github.com/apache/druid/issues/12399#issuecomment-1092195372

   Hi @Codegass feel free to submit a PR to improve some of those things. The printing in test cases might be a relic of someone debugging some of these things. To the extent that things have to be checked as part of the test, I agree it's useful to use test assertions instead. For other things that are there for information, we could probably replace them with debug statements instead 


-- 
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-unsubscribe@druid.apache.org

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


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