Github user parthchandra commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1234#discussion_r184549758
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/store/bson/TestBsonRecordReader.java
 ---
    @@ -45,21 +47,24 @@
     import org.bson.BsonTimestamp;
     import org.bson.BsonWriter;
     import org.bson.types.ObjectId;
    -import org.junit.AfterClass;
    -import org.junit.BeforeClass;
    +import org.junit.After;
    +import org.junit.Before;
     import org.junit.Test;
     
    -public class TestBsonRecordReader extends BaseTestQuery {
    -  private static VectorContainerWriter writer;
    -  private static TestOutputMutator mutator;
    -  private static BsonRecordReader bsonReader;
    +public class TestBsonRecordReader {
    +  private BufferAllocator allocator;
    +  private VectorContainerWriter writer;
    +  private TestOutputMutator mutator;
    +  private BufferManager bufferManager;
    +  private BsonRecordReader bsonReader;
     
    -  @BeforeClass
    -  public static void setUp() {
    -    BufferAllocator bufferAllocator = getDrillbitContext().getAllocator();
    -    mutator = new TestOutputMutator(bufferAllocator);
    +  @Before
    +  public void setUp() {
    +    allocator = new RootAllocator(Long.MAX_VALUE);
    --- End diff --
    
    Any reason why you had to change to RootAllocator? Should not really be 
necessary.


---

Reply via email to