Github user HyukjinKwon commented on the issue:
    I see. That makes perfect sense in general. It is good to improve test 
coverage. Though, we might have to avoid adding test cases here and there when 
there are obviously no side-effect. Running tests already takes few hours.
    At least, I am trying to submit a PR as sensible as I can, considering all 
the comments I received before, re-reading documentation and referring other 
good PRs. I left some comments above because it seems usually not or not like 
what I was told before (at least in my experience).
    Maybe we should mention it somewhere like wiki and make this official 
because I guess others might have a very different view for this. We may say, 
"please at least run them manually and then include the results in the PR 

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to