ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sid Wagle <swa...@hortonworks.com>
Subject Re: Review Request 43559: Add blueprint support for drpc.servers and mapreduce.job.hdfs-servers
Date Sat, 13 Feb 2016 18:23:49 GMT

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


Ship it!




Ship It!

- Sid Wagle


On Feb. 13, 2016, 4:43 p.m., Sumit Mohanty wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43559/
> -----------------------------------------------------------
> 
> (Updated Feb. 13, 2016, 4:43 p.m.)
> 
> 
> Review request for Ambari, Mahadev Konar, Robert Nettleton, and Sid Wagle.
> 
> 
> Bugs: AMBARI-15035
>     https://issues.apache.org/jira/browse/AMBARI-15035
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> drpc.servers is a property in storm yaml config file that captures the drpc server hostname.
During blueprint based deployments this can be specified by the user but some code changes
are needed to have the blueprint config processor resolve the value to actual hostname.
> 
> mapreduce.job.hdfs-servers support need to be added to resolve to NN host.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java
7fb2592 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
9c76e8a 
> 
> Diff: https://reviews.apache.org/r/43559/diff/
> 
> 
> Testing
> -------
> 
> Tested manually and added unit tests for these two new properties.
> 
> 
> Thanks,
> 
> Sumit Mohanty
> 
>


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