----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34071/#review85208 -----------------------------------------------------------
Ship it! One minor comment. exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BasicServer.java <https://reviews.apache.org/r/34071/#comment136721> I would just say 'Server type' instead of 'server type' - Parth Chandra On May 20, 2015, 5:49 p.m., Daniel Barclay wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34071/ > ----------------------------------------------------------- > > (Updated May 20, 2015, 5:49 p.m.) > > > Review request for drill, abdelhakim deneche, Mehant Baid, and Parth Chandra. > > > Bugs: DRILL-3034 > https://issues.apache.org/jira/browse/DRILL-3034 > > > Repository: drill-git > > > Description > ------- > > Applied UserException to can't-bind-to-port error. [BasicServer] > Added specific handling of UserException (above case or other) in > SQLException wrapping. [DrillConnectionImpl] > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BasicServer.java > a148436 > exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillConnectionImpl.java > 30279e6 > > Diff: https://reviews.apache.org/r/34071/diff/ > > > Testing > ------- > > Manually tested in SQLLine and debugger. > > Ran regular tests; no new failures. > > > Thanks, > > Daniel Barclay > >
