ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Newton Alex" <na...@pivotal.io>
Subject Re: Review Request 38703: AMBARI-13156 - Display HAWQ General Section in HAWQ config screen
Date Wed, 30 Sep 2015 21:04:11 GMT


> On Sept. 30, 2015, 8:56 p.m., jun aoki wrote:
> > ambari-web/app/data/HDP2.3/site_properties.js, line 579
> > <https://reviews.apache.org/r/38703/diff/1/?file=1083690#file1083690line579>
> >
> >     Not knowing too much how this collaborate with the helper below, but hawqmaster_host
and hawqstandby_host are missing from here. Is it a problem?

The hawq_master_address_host is the actual hawqmaster_host (the display name should be saying
HAWQ Master). The site_property.js contains the exact name as in the config file(in this case
hawq-site.xml)


- Newton


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


On Sept. 24, 2015, 1:12 a.m., Newton Alex wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38703/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2015, 1:12 a.m.)
> 
> 
> Review request for Ambari.
> 
> 
> Bugs: AMBARI-13156
>     https://issues.apache.org/jira/browse/AMBARI-13156
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Display HAWQ General Section in HAWQ config screen along with substituted hostname values
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/data/HDP2.3/site_properties.js 10390ac 
>   ambari-web/app/models/stack_service.js 5ba528a 
>   ambari-web/app/utils/configs/config_property_helper.js 9b528dd 
> 
> Diff: https://reviews.apache.org/r/38703/diff/
> 
> 
> Testing
> -------
> 
> 6761 tests complete (16 seconds)
> 94 tests pending
> 
> [INFO]
> [INFO] --- apache-rat-plugin:0.11:check (default) @ ambari-web ---
> [INFO] 51 implicit excludes (use -debug for more details).
> [INFO] Exclude: .idea/**
> [INFO] Exclude: package.json
> [INFO] Exclude: public/**
> [INFO] Exclude: public-static/**
> [INFO] Exclude: app/assets/**
> [INFO] Exclude: vendor/**
> [INFO] Exclude: node_modules/**
> [INFO] Exclude: node/**
> [INFO] Exclude: npm-debug.log
> [INFO] 1207 resources included (use -debug for more details)
> [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 1207
licence.
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 33.238 s
> [INFO] Finished at: 2015-09-23T17:58:27-08:00
> [INFO] Final Memory: 10M/308M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Newton Alex
> 
>


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