b5224-b255-404a-bd24-1b3ce5f79649__AMBARI-15331.patch
Thanks,
Shantanu Mundkur
uot; cardinlaity for master components then
> > lets not show any master host component with "ALL" cardinality on "Assign
> > Masters page". It will be confusing if end-user deselects host components
> > on "Assign Master" page and even after that amba
uot; cardinlaity for master components then
> > lets not show any master host component with "ALL" cardinality on "Assign
> > Masters page". It will be confusing if end-user deselects host components
> > on "Assign Master" page and even after that amba
imental and so thought a default of False might have been preferable
but Ambari chooses it as True out of the box. Thoughts?
Thanks.
- Shantanu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews
automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44525/#review122808
-------
On March 9, 2016, 8:59 p.m., Shantanu Mundkur wrote:
>
> ---
> This is an automatically
; >
> > In any case, after https://issues.apache.org/jira/browse/AMBARI-15170,
> > these properties are not used in the ambari-metrics-collector startup
> > script.
>
> Shantanu Mundkur wrote:
> Aravindan,
>
> That is right. We don't really ne
mail. To reply, visit:
https://reviews.apache.org/r/44525/#review122828
---
On March 9, 2016, 8:59 p.m., Shantanu Mundkur wrote:
>
> ---
> This is an automatically generated
onfigurationProcessor.java
f5e7578
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
68d5755
Diff: https://reviews.apache.org/r/45056/diff/
Testing
---
1) ambari-server unit tests
Results :
Tests run: 3968, Failures: 0, Errors: 0, Skipped: 33
2) Added unit test to
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
3) Deployed clusters 1-5 nodes specifying "config_recommendation_strategy" :
"ONLY_STACK_DEFAULTS_APPLY"
Thanks,
Shantanu Mundkur
created? Thank you.
- Shantanu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45056/#review124404
---
On March 18, 2016, 10:31 p.m., Shantanu Mundkur wrote:
>
> --
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45056/#review124404
-------
On March 18, 2016, 10:31 p.m., Shantanu Mundkur wrote:
>
> ---
>
test_missing_shell_argument (TestUserResource.TestUserResource) ... FAIL
Thanks,
Shantanu Mundkur
t
Am I missing something?
- Shantanu
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46420/#review129764
---
On April 20,
views.apache.org/r/46420/#review129792
---
On April 20, 2016, 2:57 a.m., Shantanu Mundkur wrote:
>
> ---
> This is an automatically generated e-mail. To rep
> On April 28, 2016, 9:17 p.m., Shantanu Mundkur wrote:
> > Bob, Jayush,
> >
> > Thanks for reviewing this.
> >
> > Your review comments for the code changes are valid and Amruta will make
> > changes or respond to those.
> >
> > I ha
you!
- Shantanu Mundkur
On March 30, 2016, 10:19 p.m., Amruta Borkar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
/diff/
Testing
---
1) Tried to add a custom service with cardinality as ALL for a MASTER component
(that was previously the failing case).
2) Unit tests:
Under ambari-web:
mvn clean test
Results:
28946 tests complete (37 seconds)
154 tests pending
Thanks,
Shantanu Mundkur
iew141328
---
On July 6, 2016, 7:20 p.m., Shantanu Mundkur wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
17 matches
Mail list logo
