ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jun aoki" <jun.aoki....@gmail.com>
Subject Re: Review Request 42173: Update hawq configs to remove hawq_standby_address_host on single node clusters
Date Thu, 14 Jan 2016 00:13:06 GMT


> On Jan. 12, 2016, 7:49 p.m., Jayush Luniya wrote:
> > @Jaimin can you review these changes?
> 
> bhuvnesh chaudhary wrote:
>     @Jaimin/Richard Z. - Please let me know for any comments so that we can get it fixed,
we are planning to push it to 2.2 as it blocks single node deploy. Thank you.

@jayush, we've waited more than a day and haven't heard anything back. We will go ahead and
commit it.


- jun


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


On Jan. 12, 2016, 1:08 a.m., bhuvnesh chaudhary wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42173/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2016, 1:08 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, jun aoki, Jayush Luniya,
Oleksandr Diachenko, Richard Zang, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-14599
>     https://issues.apache.org/jira/browse/AMBARI-14599
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Update hawq configs to remove hawq_standby_address_host on single node clusters.
> Currently, its available by default in templates, so is picked up by UI and is pushed
to the filesystem during installation. On single node cluster, this property is not required
to be displayed on UI and should not be pushed to filesystem.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/wizard/step7_controller.js 42b5b87 
>   ambari-web/test/controllers/wizard/step7_test.js a6a5fab 
> 
> Diff: https://reviews.apache.org/r/42173/diff/
> 
> 
> Testing
> -------
> 
> yes.
>   22965 tests complete (19 seconds)
>   163 tests pending
>   
>   [INFO] BUILD SUCCESS
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>


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