----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26653/#review56868 -----------------------------------------------------------
Ship it! Ship It! - Hanifi Gunes On Oct. 13, 2014, 5:39 p.m., Parth Chandra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26653/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2014, 5:39 p.m.) > > > Review request for drill, Hanifi Gunes and Steven Phillips. > > > Repository: drill-git > > > Description > ------- > > Bitservers would allocte many FragmentManagers, use only one and not > explicitly close the ones that were not used. > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/memory/Accountor.java > bd64640 > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/control/WorkEventBus.java > 5c126e1 > > exec/java-exec/src/test/java/org/apache/drill/exec/memory/TestAllocators.java > de54f6a > > Diff: https://reviews.apache.org/r/26653/diff/ > > > Testing > ------- > > Reproduced case with TPCH-07; system would run out of memory after two or > three runs. After the fix, ran the tpch-07 query 15 times without running out. > > > Thanks, > > Parth Chandra > >
