----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33405/#review81201 -----------------------------------------------------------
ambari-server/conf/windows/ambari.properties <https://reviews.apache.org/r/33405/#comment131451> I don't think this looks right; you're restricting the main connection pool to 1 connection. Also, we should just use sensible defaults in Configuration and now have weirdo values here unless they need to be tuned. - Jonathan Hurley On April 22, 2015, 2:29 p.m., Florian Barca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33405/ > ----------------------------------------------------------- > > (Updated April 22, 2015, 2:29 p.m.) > > > Review request for Ambari, Artem Baranchuk, Eugene Chekanskiy, Ivan Kozlov, > Jonathan Hurley, Jayush Luniya, and Sid Wagle. > > > Bugs: AMBARI-10637 > https://issues.apache.org/jira/browse/AMBARI-10637 > > > Repository: ambari > > > Description > ------- > > Split the Eclipselink sequence connection pool away from the main one. > > > Diffs > ----- > > ambari-server/conf/windows/ambari.properties > 58c8d9feb5ae48a58e26c605c78897ee04819dc4 > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > cd2bafd4188ff14bd22952021d32d0e5abc4e4c5 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > a57a1507a13aa9dc500e06d8e1b23459e0342f79 > > ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java > 184c8dbaf944517f0ba6643fe1a2d16d244c24db > ambari-server/src/main/resources/META-INF/persistence.xml > b864c591796967f7471e65ebbc995f8aabd2bed5 > > ambari-server/src/test/java/org/apache/ambari/server/configuration/ConfigurationTest.java > 63447cad745ea04901671ec66c10716ef5c3fe5c > > Diff: https://reviews.apache.org/r/33405/diff/ > > > Testing > ------- > > mvn clean install > > deployed in a local VM cluster -> no repro > > > Thanks, > > Florian Barca > >
