> On Dec. 21, 2016, 5:24 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java, > > line 30 > > <https://reviews.apache.org/r/54935/diff/1/?file=1590324#file1590324line30> > > > > unused import > > Dmytro Sen wrote: > The patch was initially developed and tested for branch-2.5 and here is a > patch for trunk. I'll fix this, thank you.
Thanks! - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54935/#review159824 ----------------------------------------------------------- On Dec. 21, 2016, 5:55 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54935/ > ----------------------------------------------------------- > > (Updated Dec. 21, 2016, 5:55 p.m.) > > > Review request for Ambari, Sumit Mohanty, Sid Wagle, and Vitalyi Brodetskyi. > > > Bugs: AMBARI-19268 > https://issues.apache.org/jira/browse/AMBARI-19268 > > > Repository: ambari > > > Description > ------- > > After upgrade from 2.0.0 hadoop-env doesn't contain: > > {% if is_datanode_max_locked_memory_set %} > # Fix temporary bug, when ulimit from conf files is not picked up, without > full relogin. > # Makes sense to fix only when runing DN as root > if [ "$command" == "datanode" ] && [ "$EUID" -eq 0 ] && [ -n > "$HADOOP_SECURE_DN_USER" ]; then > ulimit -l {{datanode_max_locked_memory}} > fi > {% endif %} > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > 7a385f3 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java > 8568a19 > > Diff: https://reviews.apache.org/r/54935/diff/ > > > Testing > ------- > > Unit test passed > > > Thanks, > > Dmytro Sen > >
