> On July 6, 2015, 9:27 p.m., Alejandro Fernandez wrote: > > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line > > 682 > > <https://reviews.apache.org/r/35823/diff/1-2/?file=991184#file991184line682> > > > > I meant "if (webAddressPort != null)"
hello Alenjandro, The webAddressPort value returned from the getWebAddressPort function could be an empty string instead of null, meaning somewhere in the code we have to check for falsy conditions. I added the falsy check into the getWebAddressPort function so that it either returns a non-empty result or null. I also updated the setSpecificResourceMangerConfigs function to use "if (webAddressPort != null)" that you asked. Please take a look at the latest patch file for details. Thank you. > On July 6, 2015, 9:27 p.m., Alejandro Fernandez wrote: > > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line > > 713 > > <https://reviews.apache.org/r/35823/diff/1-2/?file=991184#file991184line713> > > > > trim() is not supported in IE 8, should probably use regex to remove > > leading and trailing spaces. hello Alenjandro, yes, I updated the code to use a regular expression to trim the leading and trailing spaces. Thanks. - Di ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35823/#review90573 ----------------------------------------------------------- On July 6, 2015, 9:25 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35823/ > ----------------------------------------------------------- > > (Updated July 6, 2015, 9:25 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Yusaku Sako. > > > Bugs: AMBARI-12107 > https://issues.apache.org/jira/browse/AMBARI-12107 > > > Repository: ambari > > > Description > ------- > > This jira is to add some UI logic to update two more RM configuration > properties when user moves the RM (when YARN RM HA enabled) from one node to > another via Ambari web UI. > > > I have a 3 node cluster, node1, node2, and node3. > > 1. I enabled RM HA, where the node1 and node 2 are the RM active and standby > nodes respectively. > 2. I then moved RM from node 1 to node 3 via the "Move ResourceManager" menu > item on the UI. The move finished successfully. > 3. I noticed yarn.resourcemanager.hostname.rm1 was changed from node1 to > node3 correctly. yarn.resourcemanager.webapp.address.rm1 however still had > value "node1:8088", yarn.resourcemanager.webapp.https.address.rm1 still had > value "node1:8090" > > It seems to me that > ambari-web\app\controllers\main\service\reassign\step4_controller.js only > updates yarn.resourcemanager.hostname.rm1(2) when moving the RM. It should > also update the web app http and https values. > > > Diffs > ----- > > ambari-web/app/controllers/main/service/reassign/step4_controller.js > edc9b8f > ambari-web/test/controllers/main/service/reassign/step4_controller_test.js > 0713abf > > Diff: https://reviews.apache.org/r/35823/diff/ > > > Testing > ------- > > Updated JS unit tests > Manually patch my cluster, then enabled YARN RM HA, then moved RM from node 1 > to node 3, noticed this time the yarn.resourcemanager.webapp.address.rm1 and > yarn.resourcemanager.webapp.https.address.rm1 were updated to contain node1 > to contain node3. > > > Thanks, > > Di Li > >
