----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26676/#review56505 -----------------------------------------------------------
Ship it! Just minor nits. ambari-server/src/main/java/org/apache/ambari/server/view/ViewContextImpl.java <https://reviews.apache.org/r/26676/#comment96874> Is this a big deal enough to rethrow? What happens from a SystemException from here - is anything in an invalid state? ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java <https://reviews.apache.org/r/26676/#comment96873> Minor, but method name starting with capitalized? (Times 3 for each overloaded) - Nate Cole On Oct. 13, 2014, 11:34 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26676/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2014, 11:34 p.m.) > > > Review request for Ambari, Nate Cole and Srimanth Gunturi. > > > Bugs: AMBARI-7768 > https://issues.apache.org/jira/browse/AMBARI-7768 > > > Repository: ambari > > > Description > ------- > > Masked view param is not encoded when the view-instance gets created. Due to > this, the decoded value is being decoded again, resulting in garbled values. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ViewInstanceResourceProvider.java > 9a48b41 > > ambari-server/src/main/java/org/apache/ambari/server/view/ViewContextImpl.java > 0254113 > ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java > b1cec01 > > ambari-server/src/test/java/org/apache/ambari/server/view/ViewRegistryTest.java > e6a20b3 > > Diff: https://reviews.apache.org/r/26676/diff/ > > > Testing > ------- > > Unit tests updated. > > All tests pass. > > > Thanks, > > Tom Beerbower > >
