Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1045#discussion_r162250110
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/managed/TestSorter.java
 ---
    @@ -59,7 +59,7 @@
     @Category(OperatorTest.class)
     public class TestSorter extends DrillTest {
     
    -  public static OperatorFixture fixture;
    +  public volatile static OperatorFixture fixture;
    --- End diff --
    
    Why must this be volatile? This test runs in only one thread. The allocator 
that the fixture provides works in only one thread...


---

Reply via email to