-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63516/
-----------------------------------------------------------
(Updated Nov. 9, 2017, 11:58 a.m.)
Review request for Ambari, Jonathan Hurley and Nate Cole.
Changes
-------
Current patch
Bugs: AMBARI-22353
https://issues.apache.org/jira/browse/AMBARI-22353
Repository: ambari
Description
-------
Legacy/ancient ResourceProviders use the {{properties.json}} file to govern
which properties can be used with the provider. This seems like excessive
decoupling without any benefit and usually leads to runtime errors when new or
removed properties are forgotten.
This file should be removed and the providers should be registering the known
properties on their own.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/ResourceProviderFactory.java
711ae10f7e
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractAuthorizedResourceProvider.java
a24c6394ac
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java
1dc0841d19
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractDRResourceProvider.java
953a2a7c77
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ActionResourceProvider.java
4e618e7ff6
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ActiveWidgetLayoutResourceProvider.java
389f0b2bf2
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertDefinitionResourceProvider.java
32f1fd8f4f
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertGroupResourceProvider.java
cf4e7475b3
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertHistoryResourceProvider.java
8d716eabe0
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertNoticeResourceProvider.java
8ffd596cfa
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertResourceProvider.java
c71651e5ff
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertTargetResourceProvider.java
43ee7fe7c7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AmbariConfigurationResourceProvider.java
4f4cc7070d
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java
73d24908ea
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java
a2a49d7cb5
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterKerberosDescriptorResourceProvider.java
2678d55975
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java
54c83600b7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
1a755590b7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ConfigGroupResourceProvider.java
737bfa4d46
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ConfigurationResourceProvider.java
0f260854f8
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CredentialResourceProvider.java
79254c6d89
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/DefaultProviderModule.java
c3758b3f5e
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ExtensionLinkResourceProvider.java
db904bf714
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ExtensionResourceProvider.java
53e44e1888
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ExtensionVersionResourceProvider.java
4aa357c0e3
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/FeedResourceProvider.java
f9d75fd353
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/GroupPrivilegeResourceProvider.java
f9f838fe22
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/GroupResourceProvider.java
f051651e12
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostComponentProcessResourceProvider.java
2a25bc4d02
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostComponentResourceProvider.java
6708560969
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostKerberosIdentityResourceProvider.java
bfaf7b4a4f
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostResourceProvider.java
5373ffe31c
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProvider.java
ba5fccc1dc
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/InstanceResourceProvider.java
d3a695ba08
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/JobResourceProvider.java
e70c3674b4
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/KerberosDescriptorResourceProvider.java
93013415b4
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/LdapSyncEventResourceProvider.java
4badc9c472
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/LoggingResourceProvider.java
d6afe7b22b
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/MemberResourceProvider.java
d9d56eee8c
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/OperatingSystemResourceProvider.java
174fdf60a0
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/PrivilegeResourceProvider.java
fabce6b379
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/QuickLinkArtifactResourceProvider.java
073ec60b27
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ReadOnlyResourceProvider.java
505ec637ed
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RecommendationResourceProvider.java
70ed3dc09f
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RemoteClusterResourceProvider.java
0b2d2b3f52
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
81f283c55b
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestScheduleResourceProvider.java
4e25369a6e
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RoleAuthorizationResourceProvider.java
78eb80e8d1
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RootServiceComponentResourceProvider.java
3925aebdae
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RootServiceHostComponentResourceProvider.java
c37bb2efe7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RootServiceResourceProvider.java
b24a94aee0
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceConfigVersionResourceProvider.java
e7dbbc0389
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/SettingResourceProvider.java
5235ef4fb1
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackAdvisorResourceProvider.java
7035f0a148
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackArtifactResourceProvider.java
6bb421b67b
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationDependencyResourceProvider.java
366b3ef052
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProvider.java
6f5b0f8c02
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackDependencyResourceProvider.java
8b69c6a312
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProvider.java
819507bffd
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackResourceProvider.java
ead4f6df2b
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackServiceComponentResourceProvider.java
b9661bb409
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackServiceResourceProvider.java
8727741d37
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackVersionResourceProvider.java
094c75bbb8
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java
6a326b5338
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TargetClusterResourceProvider.java
d0c37f5a9e
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskAttemptResourceProvider.java
3af7bb15e5
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java
0b4fcd9633
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ThemeArtifactResourceProvider.java
fc7cfe7f91
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
e6aafc1ec5
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeItemResourceProvider.java
36eb3bebf1
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
a1ec98afe7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeSummaryResourceProvider.java
94b2b4a981
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UserAuthorizationResourceProvider.java
95550f0299
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UserPrivilegeResourceProvider.java
614f7abda1
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UserResourceProvider.java
c5c36e9942
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ValidationResourceProvider.java
b78c64af53
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/VersionDefinitionResourceProvider.java
f94ac32f17
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ViewInstanceResourceProvider.java
9562782b01
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ViewResourceProvider.java
7179b335c2
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ViewURLResourceProvider.java
cb506be1bf
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetLayoutResourceProvider.java
8c39fc4057
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetResourceProvider.java
03fd5d2359
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WorkflowResourceProvider.java
13cb8a4768
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/PropertyHelper.java
893978047c
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariMetricSinkImpl.java
5c5801d451
ambari-server/src/main/resources/key_properties.json 5d76062d87
ambari-server/src/main/resources/properties.json 1d12f83b3f
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProviderTest.java
a127de6539
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractDRResourceProviderTest.java
38e6d366a1
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractResourceProviderTest.java
7165e48d70
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ActionResourceProviderTest.java
ef56f9d3e7
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ActiveWidgetLayoutResourceProviderTest.java
5221ecf16b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintResourceProviderTest.java
f53441182b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java
833be5d117
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterKerberosDescriptorResourceProviderTest.java
ac2847590d
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java
4a808931ac
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProviderTest.java
984010650c
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ConfigGroupResourceProviderTest.java
78f79ead9c
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ConfigurationResourceProviderTest.java
785ea2388b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CredentialResourceProviderTest.java
89ed022da0
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ExtensionResourceProviderTest.java
5b6e830d6e
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/FeedResourceProviderTest.java
5cf9b056ea
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/GroupResourceProviderTest.java
bbfa2d8503
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostComponentProcessResourceProviderTest.java
5a7460d4c1
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostComponentResourceProviderTest.java
7e32c15991
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostResourceProviderTest.java
fd28081944
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProviderTest.java
6c95c53e5b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/InstanceResourceProviderTest.java
ba6793989e
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/JMXHostProviderTest.java
e19ad29453
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/JobResourceProviderTest.java
6fd1a238ae
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/KerberosDescriptorResourceProviderTest.java
ca8127e42f
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/MemberResourceProviderTest.java
39ba4b9fe3
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestResourceProviderTest.java
c0695b12d6
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestScheduleResourceProviderTest.java
6d4f72d9ad
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RootServiceComponentResourceProviderTest.java
222340b4e7
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RootServiceHostComponentResourceProviderTest.java
ce3656c4ab
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RootServiceResourceProviderTest.java
96bc7c9674
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackAdvisorResourceProviderTest.java
05232eac49
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackArtifactResourceProviderTest.java
21df88d545
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackConfigurationDependencyResourceProviderTest.java
7c88852f48
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackConfigurationResourceProviderTest.java
9c48ad0d23
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackDependencyResourceProviderTest.java
f35015348f
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackLevelConfigurationResourceProviderTest.java
e9742bdfb1
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackResourceProviderTest.java
4281e3372e
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackServiceResourceProviderTest.java
2a2ef0b0b7
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TargetClusterResourceProviderTest.java
492c012c7e
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java
365d408ff6
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TestIvoryProviderModule.java
90f10118d4
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
d6b1ab3766
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UserResourceProviderDBTest.java
c4f0f349fb
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UserResourceProviderTest.java
9efad8a54d
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ValidationResourceProviderTest.java
c639d1fcdc
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/WorkflowResourceProviderTest.java
337dc466da
Diff: https://reviews.apache.org/r/63516/diff/2/
Changes: https://reviews.apache.org/r/63516/diff/1-2/
Testing
-------
tests do not pass yet, also on deploy get errors in some places.
Fixing mistakes
Thanks,
Dmitro Lisnichenko
