ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebastian Toader" <stoa...@hortonworks.com>
Subject Re: Review Request 41213: Storm 'nimbus.seeds' property not handled correctly in Blueprint processor resulting Storm not being able to start.
Date Thu, 10 Dec 2015 18:48:07 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41213/
-----------------------------------------------------------

(Updated Dec. 10, 2015, 7:48 p.m.)


Review request for Ambari, Oliver Szabo and Robert Nettleton.


Changes
-------

Fix suffix handling.


Bugs: AMBARI-14325
    https://issues.apache.org/jira/browse/AMBARI-14325


Repository: ambari


Description
-------

MultipleHostTopologyUpdater handled only yaml type of config properties where the original
value was enclosed by ['']. This has been extended to handle also property value enclosed
only by [] (no simple quote)


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java
457b78b 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
033fe10 

Diff: https://reviews.apache.org/r/41213/diff/


Testing
-------

Manual testing for both single and multi node scenarios.


Unit tests in progress.


Thanks,

Sebastian Toader


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message