[ 
https://issues.apache.org/jira/browse/LUCENE-5622?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13976653#comment-13976653
 ] 

Dawid Weiss commented on LUCENE-5622:
-------------------------------------

I'll look at these.

> This is important to automatically add when using -Dtestcase, some developers 
> like to debug with SOP.

Isn't this the purpose of tests.verbose? I agree with what you're saying about 
SOP and I'm sort of worried that people will hate this feature (and me, 
indirectly...).

> Fail tests if they print, and tests.verbose is not set
> ------------------------------------------------------
>
>                 Key: LUCENE-5622
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5622
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Assignee: Dawid Weiss
>         Attachments: LUCENE-5622.patch, LUCENE-5622.patch, LUCENE-5622.patch
>
>
> Some tests print so much stuff they are now undebuggable (see LUCENE-5612).
> I think its bad that the testrunner hides this stuff, we used to stay on top 
> of it. Instead, whne tests.verbose is false, we should install a printstreams 
> (system.out/err) that fail the test instantly because they are noisy. 
> This will ensure that our tests don't go out of control.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to