> On Nov. 20, 2015, 1:27 a.m., Alejandro Fernandez wrote: > > Ship It!
Pushed to trunk, commit 3d3af0bce733a13b81c6c3752648e110f976c441 - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40446/#review107286 ----------------------------------------------------------- On Nov. 19, 2015, 10:14 a.m., Sebastian Toader wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40446/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2015, 10:14 a.m.) > > > Review request for Ambari, Alejandro Fernandez and Sid Wagle. > > > Bugs: AMBARI-13951 > https://issues.apache.org/jira/browse/AMBARI-13951 > > > Repository: ambari > > > Description > ------- > > If a property in stack definition has property type which is unknown to > Ambari (know property types are listed in PropertyInfo.PropertType) Jaxb will > deserialize this into null reference. This null reference will cause later > service add operations to fail. > > To overcome this we just simply skip unknown property types as property types > are not mandatory for the properties. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java > bc90a8c > > ambari-server/src/test/java/org/apache/ambari/server/state/PropertyInfoTest.java > 31cf80b > > Diff: https://reviews.apache.org/r/40446/diff/ > > > Testing > ------- > > Manual testing: > - Removed "TEXT" property type form PropertyInfo.PropertType enum (this way > making "TEXT" unkown to Ambari) > - Deployed a cluster with 3 nodes > - Added Ranger service to the cluster as Ranger service has properties with > property type "TEXT" > - Verified that Ranger service installed successfully. > > Unit tests: > Results : > > Tests run: 3414, Failures: 0, Errors: 0, Skipped: 27 > > > Thanks, > > Sebastian Toader > >
