With all due respect, I did start a review. Is something broken?

- Paul

> On Oct 18, 2017, at 3:36 PM, julianhyde <g...@git.apache.org> wrote:
> 
> Github user julianhyde commented on a diff in the pull request:
> 
>    https://github.com/apache/drill/pull/984#discussion_r145561518
> 
>    --- Diff: 
> exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java ---
>    @@ -584,11 +492,14 @@ public static void defineWorkspace(Drillbit 
> drillbit, String pluginName,
>       public static final String EXPLAIN_PLAN_TEXT = "text";
>       public static final String EXPLAIN_PLAN_JSON = "json";
> 
>    -  public static FixtureBuilder builder() {
>    -    FixtureBuilder builder = new FixtureBuilder()
>    +  public static FixtureBuilder builder(DirTestWatcher dirTestWatcher) {
>    --- End diff --
> 
>    Jeez Paul, please start a review rather than making single review 
> comments. I just got 39 emails from you, and so did everyone else on 
> dev@drill.
> 
> 
> ---

Reply via email to