ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eugene Chekanskiy" <echekans...@hortonworks.com>
Subject Re: Review Request 39687: Allow use passwords references in custom actions
Date Tue, 03 Nov 2015 10:50:18 GMT


> On Nov. 2, 2015, 9:43 p.m., Robert Nettleton wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java,
line 2937
> > <https://reviews.apache.org/r/39687/diff/3/?file=1114085#file1114085line2937>
> >
> >     Why was the "synchronized" keyword removed?  
> >     
> >     It looks like this method was moved into the StackInfo class, but I would imagine
that the concurrency requirements would be the same for protecting this data from concurrent
access.

Cache moved to StackInfo, Collection.synchronizedMap used instead of synchronized.


- Eugene


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


On Nov. 2, 2015, 5:24 p.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39687/
> -----------------------------------------------------------
> 
> (Updated Nov. 2, 2015, 5:24 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Nettleton, and Sid
Wagle.
> 
> 
> Bugs: AMBARI-13582
>     https://issues.apache.org/jira/browse/AMBARI-13582
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Added ability to use secret references in custom actions; /api/v1/blueprints and /api/v1/clusters/cl?format=blueprint
returns masked passwords.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java
d834731 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
5f432ef 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java 625ebc6

>   ambari-server/src/main/java/org/apache/ambari/server/controller/ConfigurationResponse.java
3ed9306 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java
fa355fa 
>   ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java def33f0

>   ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java e3ac3e0 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
279b31f 
>   ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java
9e8f163 
>   ambari-server/src/main/java/org/apache/ambari/server/utils/SecretReference.java 2b1aeae

>   ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
c6170e4 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintResourceProviderTest.java
369bf02 
>   ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintImplTest.java
2199247 
>   ambari-server/src/test/resources/stacks/HDP/2.0.6/services/YARN/configuration/yarn-site.xml
1c02e86 
> 
> Diff: https://reviews.apache.org/r/39687/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>


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