Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1045#discussion_r162517423
--- 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 --
Strange... So, if having volatile does solve a problem, just leave it in,
but maybe add a comment that includes you note above so that we know what's
what in the future.
---
