> On Dec. 11, 2017, 10:23 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql > > Line 42 (original) > > <https://reviews.apache.org/r/64478/diff/1/?file=1911954#file1911954line42> > > > > Where are we setting this value? I saw we removed it from the > > StackManager code. So, in UI deployment it might fail.
Mistakenly left there. Will remove. - Balázs Bence ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64478/#review193429 ----------------------------------------------------------- On Dec. 9, 2017, 12:19 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64478/ > ----------------------------------------------------------- > > (Updated Dec. 9, 2017, 12:19 p.m.) > > > Review request for Ambari, Balázs Bence Sári, Jayush Luniya, Madhuvanthi > Radhakrishnan, Robert Nettleton, and Swapan Shridhar. > > > Bugs: AMBARI-22253, AMBARI-22297 and AMBARI-22325 > https://issues.apache.org/jira/browse/AMBARI-22253 > https://issues.apache.org/jira/browse/AMBARI-22297 > https://issues.apache.org/jira/browse/AMBARI-22325 > > > Repository: ambari > > > Description > ------- > > Implementation of Blueprint V2 API POST that allows cluster creation. > Co-authored by Balázs Bence Sári and Attila Doroszlai. > > Includes minor fixes for: > > * AMBARI-22196 -- yum repo file contains parsing errors due to initial > whitespace > * AMBARI-22249 -- AmbariContext.serviceGroupDependencyResourceProvider > initialization > * AMBARI-22164 -- wrong assignment due to leftover parameter name > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/ObjectNotFoundException.java > 75c9f3b5d50102f3beeeaa8041178cc08e22e241 > > ambari-server/src/main/java/org/apache/ambari/server/ParentObjectNotFoundException.java > 15bd7cb6bbe5026bf0a5ae9d3fc0bec575137643 > > ambari-server/src/main/java/org/apache/ambari/server/StackAccessException.java > b8bfff3991a4e49514801dbb480958941cff01ac > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > fd43edfa88339cd94686ddc13822b36ab7c74d6f > > ambari-server/src/main/java/org/apache/ambari/server/checks/ServiceCheckValidityCheck.java > 2582c71eb95255f0f8081e90473183243f12cdd0 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 59e6622f708e7d8d27c9e28ce2ce58f014a4a5dc > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > c88b23c8af374dd9bd695b70560079871f3366fd > > ambari-server/src/main/java/org/apache/ambari/server/controller/ConfigurationRequest.java > d26cf16b59897573173ea258133a1c1b4e6f31f1 > > ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java > f2ff181da0327a82adb67e1405ab9ae238b5c4d4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/DeleteIdentityHandler.java > cc84ce24b5e769257ac0e8744a93e3e236bb04e3 > > ambari-server/src/main/java/org/apache/ambari/server/controller/RepositoryResponse.java > 6d0f3fbd4288d31c3e077445e20c3b2bac696930 > > ambari-server/src/main/java/org/apache/ambari/server/controller/ServiceComponentHostRequest.java > ea082c66e128a30368d6be195f3ee270a08e8518 > > ambari-server/src/main/java/org/apache/ambari/server/controller/StackV2.java > 0b0329c31bb5800eb32ba4b437bd57a41a618dbe > > ambari-server/src/main/java/org/apache/ambari/server/controller/StackV2Factory.java > 784b3683ba9d3909bb9e6d900e4175b6a00aec9a > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java > 5104354fce7701fe023723c19d56d25e7adf5219 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BaseClusterRequest.java > a38f478ed05ec88816e6363b31aa102a4a9d353f > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintV2ResourceProvider.java > 6d2c4f06f5c75fbdf9e81c2d711440862f375bb9 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java > cc0cea377233931bf65943f792f6a7e11b074f06 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ExportBlueprintRequest.java > 565369b08b6eeb1a598575e448ce0c6334b6234b > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequest.java > f773fc8aac61f74988dd63a7a8b2cc30053023d2 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RootServiceComponentConfigurationResourceProvider.java > 5a6dff4144bc059ff2614fb1ee4704c0c7a64809 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ScaleClusterRequest.java > 1284c265c150aa03c00b43b364c2163ca4e564a5 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceResourceProvider.java > 38e1233308a7f423789a1961e3dd8664919f29ae > > ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/RemovableIdentities.java > cd23e834b160f889326f65f68162d5b28906e050 > > ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/UsedIdentities.java > 2f0974e95d7dc27c9b3e33fa426b0e5dc3b3922c > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StackEntity.java > c479cdd42742ebada75f57efafd764d6b2112148 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java > 6bb248e19bef39ba2edb3671d38ef815d8efb593 > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java > dc77f19d6389071e45fccb31942c12c8f49e59e0 > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java > 4008c21f8ef092e2a1ebd0d7c18ad95bc8206e37 > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java > 36918ccfef596e14a0d3e8e663fb097d065e27a4 > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java > a6c650705ad6b90834906bd3ce08149e2528f28e > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > 3b0ebaec5480c74774f9b701ea5e0f39f18f69e5 > > ambari-server/src/main/java/org/apache/ambari/server/topology/AmbariContext.java > f4137683aaad84d5a66b93b4559abd7a1fb926bc > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintFactory.java > 24b4785562c6c4998103d20aaade7c1754277a76 > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintImplV2.java > d31e9d44e7ab79d09b74171387c5ad32caf1a65e > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintV2.java > 9ca024810edbed9d9eb3ebaf2107c994db2bf1df > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintV2Factory.java > 7b228e57ad176bff43533127b840270efc115fa6 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java > 3273a4ead6bb3546c16676cd5e748a669e518a45 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopology.java > 289f053ccef23d67fa1db9dd298d53d38098ed3b > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopologyImpl.java > db3f29bda9cc2300ba135dbb7dff7903f219ca41 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ComponentV2.java > 68744d4938f4e9fb171077ef6bee98fc298509fb > > ambari-server/src/main/java/org/apache/ambari/server/topology/Configurable.java > 74308ab3d8caf5745478da2975da5c3c6429a5ba > > ambari-server/src/main/java/org/apache/ambari/server/topology/Configuration.java > 28dbbaa5af9a7321db8773d85be60ad771dd34d6 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ConfigurationFactory.java > 7f9a06f2d2b7f3b51f87304a8e51e019c7f175a8 > > ambari-server/src/main/java/org/apache/ambari/server/topology/Credential.java > 3146e2fecaf9a2e6c2c72aad3fbc69828a49eef4 > > ambari-server/src/main/java/org/apache/ambari/server/topology/HostGroupInfo.java > 7cbdd98cf349b7100bb0ffac96109ff10f959893 > > ambari-server/src/main/java/org/apache/ambari/server/topology/HostGroupV2.java > 8da24bd3c7891a1a988ba0470210f3d4ca908032 > > ambari-server/src/main/java/org/apache/ambari/server/topology/HostGroupV2Impl.java > 01f34da4b530045eb749b8955f1a737701400d19 > > ambari-server/src/main/java/org/apache/ambari/server/topology/PersistedStateImpl.java > 1374a42fa190f89b48913b98908583694c05378b > > ambari-server/src/main/java/org/apache/ambari/server/topology/SecurityConfiguration.java > 7955169da4639d1322d81b4499cc546604889e46 > ambari-server/src/main/java/org/apache/ambari/server/topology/Service.java > 317e29f17cb0b0cfdc86e41f96e6266f75c673f0 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ServiceGroup.java > 7f38b2c67a03165dd5616cf5ea7355e28f44d136 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ServiceId.java > 2d81a07ad37d9f07c7a294316887bf037b6a54d9 > ambari-server/src/main/java/org/apache/ambari/server/topology/Setting.java > 34542e13c4aa66c8e6513d03672a7049accc83b3 > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java > 3b7dcc88bacb376afc25c7970e7c1f683f0724b7 > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyRequestFactory.java > 751e2d7a8511a0745c09357dd2792ce3bf32ec8d > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyRequestFactoryImpl.java > ff79f526ea68af1123daffb054021d8bb5cce51e > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyTemplate.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyTemplateFactory.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/topology/tasks/PersistHostResourcesTask.java > 00eb706b348a8b5a7ac5516f7b8a62ece6013b13 > > ambari-server/src/main/java/org/apache/ambari/server/topology/validators/ClusterConfigTypeValidator.java > 7ac75e937d8ac3c8cc6e65d3a5f6d4838fff8bbc > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql > 84349b6d2fa41fda1ad4e7ddfa0286d4b6814f4c > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql > f9d5321e76a3152cf47f1a26ce7a4a7265ff3436 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql > 227645f1bb9ce09067d98b8048cea29010392692 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql > 333a0c3bc3c4885b887c2d4f3a679825434f5422 > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql > 120bdb60d4b9569ba8f8a7bea844f7537218b212 > ambari-server/src/main/resources/cluster-settings.xml > 70b100aba3d1378a7eb0d29c1c6dc609b42d8d89 > > ambari-server/src/test/java/org/apache/ambari/server/StackAccessExceptionTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java > f9765f2a8ac1f2f438253ad86401345c6da5e01c > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java > 6e503a6fd91e691841ba9e9681f8aea8ec448ae4 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequestTest.java > 01d69a51095936e6c10bf3c77447bf0ed65c8131 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ScaleClusterRequestTest.java > b1e52cc120683b0de42b9a4b59514fe37ba076f9 > ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java > dafc5e800a22926058b97abeed53e52cf4e42497 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterImplTest.java > d40679aea7fedf3a858cc7683ea6db7b0e4ff606 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java > 730d51cb98eef9e0b83c242b12a21ddc2deb8586 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ConcurrentServiceConfigVersionTest.java > 558b334267fd688d46b52a36537e8b893a886197 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ServiceComponentHostConcurrentWriteDeadlockTest.java > ca7a7aceb364a74eb5d2859dc69a838c1c524fdc > > ambari-server/src/test/java/org/apache/ambari/server/topology/AmbariContextTest.java > 16de535efc3c6ba3064ca2be5841b19b69038b1a > > ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintV2FactoryTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintV2ImplTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterDeployWithStartOnlyTest.java > a22cef355dd083001a373a823f138a59a28f1c47 > > ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterInstallWithoutStartOnComponentLevelTest.java > ba2118d5a95cd1e1649085640928317ff3bec6b6 > > ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterInstallWithoutStartTest.java > 8d58107618c9b6ba786ea328cd5794e9d300b128 > > ambari-server/src/test/java/org/apache/ambari/server/topology/ConfigurableTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/ConfigurationFactoryTest.java > 14614fd718b92ea4cdb17080f61a8a5b6daa5267 > > ambari-server/src/test/java/org/apache/ambari/server/topology/SecurityConfigurationTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/TopologyManagerTest.java > 4eb29b721e8250b1fc91fb4b014abfc90a291762 > > ambari-server/src/test/java/org/apache/ambari/server/topology/TopologyTemplateFactoryTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/validators/BlueprintImplV2Test.java > ad2cdbdaf60149d12e782674a95cdf8c52ab3e23 > > ambari-server/src/test/java/org/apache/ambari/server/topology/validators/RequiredConfigPropertiesValidatorTest.java > f6b4ec8cb476dc5ef9874328c17599c538163dba > > ambari-server/src/test/java/org/apache/ambari/server/utils/ResourceUtils.java > PRE-CREATION > ambari-server/src/test/resources/blueprintv2/blueprintv2.json > c5e2541ff58697f24f829878e7b3f35226c3ec80 > ambari-server/src/test/resources/blueprintv2/blueprintv2_2.json > PRE-CREATION > ambari-server/src/test/resources/blueprintv2/cluster_template_v2.json > PRE-CREATION > > ambari-server/src/test/resources/blueprintv2/cluster_template_v2_invalid_hostgroup.json > PRE-CREATION > ambari-server/src/test/resources/blueprintv2/security_config.json > PRE-CREATION > > > Diff: https://reviews.apache.org/r/64478/diff/1/ > > > Testing > ------- > > * Tested manually: created cluster via blueprint with two service groups > using HDP 3.0 mpack/stack. > * Added new unit tests, changed some existing ones. > > > Thanks, > > Attila Doroszlai > >
