iwasakims commented on a change in pull request #641: URL: https://github.com/apache/bigtop/pull/641#discussion_r429035607
########## File path: provisioner/docker/config_centos-7.yaml ########## @@ -19,6 +19,6 @@ docker: repo: "http://repos.bigtop.apache.org/releases/1.4.0/centos/7/$basearch"; distro: centos -components: [hdfs, yarn, mapreduce] +components: [hdfs, yarn, mapred] enable_local_repo: false smoke_test_components: [hdfs, yarn, mapreduce] Review comment: It would be nice if we can make it consistent between components and smoke_test_components (mapred vs. mapreduce). Can we change [the mapred label of the map defined in cluster.pp](https://github.com/apache/bigtop/blob/master/bigtop-deploy/puppet/manifests/cluster.pp#L57) to mapreduce? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
