> On March 2, 2015, 7:34 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/KdcServerConnectionVerification.java, > > line 160 > > <https://reviews.apache.org/r/31631/diff/1/?file=882274#file882274line160> > > > > Doesn't this do the timeout _work_ rather than needed to create a > > thread?
You would think so, but in some UDP cases this library doesn't honor the timeout and attempts to read data forever. I was also surprised by this. - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31631/#review74798 ----------------------------------------------------------- On March 2, 2015, 7:06 p.m., John Speidel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31631/ > ----------------------------------------------------------- > > (Updated March 2, 2015, 7:06 p.m.) > > > Review request for Ambari, Robert Levas and Robert Nettleton. > > > Bugs: AMBARI-9870 > https://issues.apache.org/jira/browse/AMBARI-9870 > > > Repository: ambari > > > Description > ------- > > The KDC connection test functionality that is used by the UI doesn't work > when the KDC server is configured to use UDP. > This is a significant issue on Ubuntu where the MIT KDC uses UDP as the > default. > The current functionality only attempts to connect via TCP. > > > Diffs > ----- > > ambari-project/pom.xml 0577bee > ambari-server/pom.xml c57a2d0 > > ambari-server/src/main/java/org/apache/ambari/server/KdcServerConnectionVerification.java > 8bfbc5f > > ambari-server/src/test/java/org/apache/ambari/server/api/rest/KdcServerConnectionVerificationTest.java > f8ec650 > > Diff: https://reviews.apache.org/r/31631/diff/ > > > Testing > ------- > > Manual Functional Tests: > - both positive and negative tests for TCP and UDP > > Unit Tests: > - added new unit tests for UDP validation > - all unit tests pass > > Results : > Tests run: 2751, Failures: 0, Errors: 0, Skipped: 15 > ... > Total run:608 > Total errors:0 > Total failures:0 > > > Thanks, > > John Speidel > >
