> On May 26, 2015, 6:04 p.m., Parth Chandra wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BasicServer.java, > > lines 201-202 > > <https://reviews.apache.org/r/34071/diff/2/?file=956682#file956682line201> > > > > I would just say 'Server type' instead of 'server type'
Did you mean to drop "sub", capitalize, or both? - Daniel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34071/#review85208 ----------------------------------------------------------- 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 > >
