----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34179/#review83660 -----------------------------------------------------------
Ship it! Ship It! - Venki Korukanti On May 13, 2015, 1:18 p.m., abdelhakim deneche wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34179/ > ----------------------------------------------------------- > > (Updated May 13, 2015, 1:18 p.m.) > > > Review request for drill and Venki Korukanti. > > > Bugs: DRILL-3046 > https://issues.apache.org/jira/browse/DRILL-3046 > > > Repository: drill-git > > > Description > ------- > > fixes a leak in DataTunnel where it didn't release the buffers in case it was > interrupted while waiting for the sending semaphore > > TestDrillbitResilience now extends DrillTest so it will time out instead of > hanging indefinitely. > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/data/DataTunnel.java > 143020f > > exec/java-exec/src/test/java/org/apache/drill/exec/server/TestDrillbitResilience.java > 3efb629 > > Diff: https://reviews.apache.org/r/34179/diff/ > > > Testing > ------- > > > Thanks, > > abdelhakim deneche > >
