You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/04/13 17:50:13 UTC

[GitHub] [druid] kanibs commented on issue #8885: Enable static imports in tests

kanibs commented on issue #8885: Enable static imports in tests
URL: https://github.com/apache/druid/issues/8885#issuecomment-613011579
 
 
   Hey @leventov, So I was going through all the attached resources in this issue- and to my understanding, I believe that we need to refactor the Unit Tests in such a way we have minimum statics and hence avoiding static imports and usages. Please correct me here if I have understood in a different way. If that is the case then fixing all the tests in a single PR would not be so easy. So can we break this PR in a more granular way to reduce the scope to a few Classes or maybe module wise, whichever is less?

----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org