[
https://issues.apache.org/jira/browse/DRILL-7402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16951315#comment-16951315
]
ASF GitHub Bot commented on DRILL-7402:
---------------------------------------
arina-ielchiieva commented on pull request #1872: DRILL-7402: Suppress batch
dumps for expected failures in tests
URL: https://github.com/apache/drill/pull/1872#discussion_r334642872
##########
File path: exec/java-exec/src/test/java/org/apache/drill/TestOperatorDump.java
##########
@@ -34,16 +41,11 @@
import org.junit.BeforeClass;
import org.junit.Test;
-import java.nio.file.Paths;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Set;
-import java.util.stream.Collectors;
-
import static org.junit.Assert.assertTrue;
+import ch.qos.logback.classic.spi.ILoggingEvent;
Review comment:
I thought static imports should be the last ones...
----------------------------------------------------------------
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:
[email protected]
> Suppress batch dumps for expected failures in tests
> ---------------------------------------------------
>
> Key: DRILL-7402
> URL: https://issues.apache.org/jira/browse/DRILL-7402
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.16.0
> Reporter: Paul Rogers
> Assignee: Paul Rogers
> Priority: Minor
>
> Drill provides a way to dump the last few batches when an error occurs.
> However, in tests, we often deliberately cause something to fail. In this
> case, the batch dump is unnecessary.
> This enhancement adds a config property, disabled in tests, that controls the
> dump activity.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
