drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Parth Chandra" <pchan...@maprtech.com>
Subject Re: Review Request 28686: For DRILL-1735, hook up JDBC connection closing, and fix follow-on resource bugs.
Date Sat, 14 Feb 2015 01:53:18 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28686/#review72477
-----------------------------------------------------------

Ship it!


Ship It!

- Parth Chandra


On Feb. 5, 2015, 7:28 p.m., Daniel Barclay wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28686/
> -----------------------------------------------------------
> 
> (Updated Feb. 5, 2015, 7:28 p.m.)
> 
> 
> Review request for drill, Mehant Baid and Parth Chandra.
> 
> 
> Bugs: DRILL-1735
>     https://issues.apache.org/jira/browse/DRILL-1735
> 
> 
> Repository: drill-git
> 
> 
> Description
> -------
> 
> This change hooks in the previously written but unused DrillHandler so that a local-mode
JDBC connection shuts down its embedded DrillBit. 
> 
> It also fixes a chain of follow-on bugs (mostly resource-deallocation omissions) exposed
by the original bug fixes.  (See the commit message for the list of bugs/fixes and other changes.)
> 
> 
> Diffs
> -----
> 
>   exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillClient.java 04b955b

>   exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScreenCreator.java
d884200 
>   exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/QueryResultHandler.java
9015a16 
>   exec/java-exec/src/main/java/org/apache/drill/exec/server/BootStrapContext.java 3da2ea9

>   exec/java-exec/src/main/java/org/apache/drill/exec/vector/accessor/SqlAccessor.java
b8480b4 
>   exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java 99c6ab8 
>   exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java 378e81a

>   exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java
a8f07b5 
>   exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java 0272b23 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/AvaticaDrillSqlAccessor.java cf5829a

>   exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillAccessorList.java 7594783 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillConnectionConfig.java 54e31b1 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillConnectionImpl.java 2c51ec0 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillCursor.java fbe611f 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillResultSet.java 88a6c6d 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/Driver.java 974e786 
>   exec/jdbc/src/main/java/org/apache/drill/jdbc/InvalidCursorStateSqlException.java PRE-CREATION

>   exec/jdbc/src/main/java/org/apache/drill/jdbc/JdbcApiSqlException.java PRE-CREATION

>   exec/jdbc/src/test/java/org/apache/drill/jdbc/DrillResultSetTest.java PRE-CREATION

>   exec/jdbc/src/test/java/org/apache/drill/jdbc/DriverTest.java PRE-CREATION 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/Bug1735ConnectionCloseTest.java
PRE-CREATION 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/Bug1735ResultSetCloseReleasesBuffersTest.java
PRE-CREATION 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestJdbcDistQuery.java b2f86ea 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestJdbcQuery.java b627c38 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestMetadataDDL.java c52eafd 
>   exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestViews.java 28fa4a7 
> 
> Diff: https://reviews.apache.org/r/28686/diff/
> 
> 
> Testing
> -------
> 
> Ran tests.
> 
> 
> Thanks,
> 
> Daniel Barclay
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message