----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34074/ -----------------------------------------------------------
Review request for drill, Chris Westin and Venki Korukanti. Repository: drill-git Description ------- [DRILL-3035](https://issues.apache.org/jira/browse/DRILL-3035): Created ControlsInjector interface to enforce method implementations + Fixes potential NPEs from site classes that use injectInterruptiblePause Diffs ----- exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScanBatch.java 6176f77 exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScreenCreator.java 76dc91c exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/mergereceiver/MergingRecordBatch.java 5d990f0 exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerDecorator.java c355070 exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java e40fe54 exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java 6961ee6 exec/java-exec/src/main/java/org/apache/drill/exec/store/pojo/PojoRecordReader.java a893da1 exec/java-exec/src/main/java/org/apache/drill/exec/testing/ControlsInjector.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/testing/ExecutionControlsInjector.java 387d300 exec/java-exec/src/main/java/org/apache/drill/exec/testing/NoOpControlsInjector.java bb13d1f exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java bf62ccb exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java d96e6d6 exec/java-exec/src/test/java/org/apache/drill/exec/testing/TestCountDownLatchInjection.java c98f54c exec/java-exec/src/test/java/org/apache/drill/exec/testing/TestExceptionInjection.java e3558a1 exec/java-exec/src/test/java/org/apache/drill/exec/testing/TestPauseInjection.java ba29c58 Diff: https://reviews.apache.org/r/34074/diff/ Testing ------- Passes regression and unit tests. Thanks, Sudheesh Katkam
