----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44858/#review123777 -----------------------------------------------------------
Ship it! This is the correct way to fix the problem. Most tests that are going to have a problem with configuring a port and then finding it already in use are using the API instead of XML and we're going to have to fix the tests individually. They need to be changed to stop using AvailablePort/AvailablePortHelper. - Bruce Schuchardt On March 15, 2016, 10:18 p.m., Jianxia Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44858/ > ----------------------------------------------------------- > > (Updated March 15, 2016, 10:18 p.m.) > > > Review request for geode, Bruce Schuchardt, Hitesh Khamesra, and Udo > Kohlmeyer. > > > Repository: geode > > > Description > ------- > > In order to avoid an already used port, use tcp port 0, so that the test will > be automatically assigned an available port. > > > Diffs > ----- > > > geode-core/src/test/java/com/gemstone/gemfire/cache30/CacheXml80DUnitTest.java > 98134ce > > Diff: https://reviews.apache.org/r/44858/diff/ > > > Testing > ------- > > precheckin > > > Thanks, > > Jianxia Chen > >
