ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmytro Sen" <d...@hortonworks.com>
Subject Re: Review Request 31373: In cluster layout recommendation, slave component cardinality of 0+ not being used
Date Wed, 25 Feb 2015 12:11:55 GMT

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

Ship it!


Ship It!

- Dmytro Sen


On Фев. 25, 2015, 10:45 д.п., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31373/
> -----------------------------------------------------------
> 
> (Updated Фев. 25, 2015, 10:45 д.п.)
> 
> 
> Review request for Ambari and Dmytro Sen.
> 
> 
> Bugs: AMBARI-9778
>     https://issues.apache.org/jira/browse/AMBARI-9778
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> We now have a slave component that has a cardinality of `0+`. This component
> in UI install wizard's host selection page does not want to be selected unless
> the user specifically checks hosts. Upto now we have not used cardinality to
> determine slave-component layout.
> 
> This should change so that when slave-component layout is being determined, it
> should ask if component-name should use cardinality for layout. Call this
> method `isComponentUsingCardinalityForLayout(componentName)`. The default-
> stack-advisor will have always return false. However some stack-version could
> return true. If so, the minimum number in the cardinality will determine the
> number of hosts for the slave-component. If it is `0+`, then 0 hosts will be
> selected. If it is `1+`, then atleast 1 host will be selected.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/stack_advisor.py 3d22d92 
> 
> Diff: https://reviews.apache.org/r/31373/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>


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