GauthamBanasandra commented on a change in pull request #3349:
URL: https://github.com/apache/hadoop/pull/3349#discussion_r708481994
##########
File path: BUILDING.txt
##########
@@ -21,38 +21,15 @@ The easiest way to get an environment with all the
appropriate tools is by means
of the provided Docker config.
This requires a recent version of docker (1.4.1 and higher are known to work).
-On Linux:
+On Linux or Mac:
Install Docker and run this command:
$ ./start-build-env.sh
-On Mac:
- First make sure Virtualbox and docker toolbox are installed.
- You can use docker toolbox as described in
http://docs.docker.com/mac/step_one/.
- $ docker-machine create --driver virtualbox \
- --virtualbox-memory "4096" hadoopdev
- $ eval $(docker-machine env hadoopdev)
- $ ./start-build-env.sh
-
-The prompt which is then presented is located at a mounted version of the
source tree
-and all required tools for testing and building have been installed and
configured.
-
Note that from within this docker environment you ONLY have access to the
Hadoop source
tree from where you started. So if you need to run
dev-support/bin/test-patch /path/to/my.patch
then the patch must be placed inside the hadoop source tree.
-
-Known issues:
-- On Mac with Boot2Docker the performance on the mounted directory is
currently extremely slow.
- This is a known problem related to boot2docker on the Mac.
- See:
- https://github.com/boot2docker/boot2docker/issues/593
- This issue has been resolved as a duplicate, and they point to a new feature
for utilizing NFS mounts
- as the proposed solution:
- https://github.com/boot2docker/boot2docker/issues/64
- An alternative solution to this problem is to install Linux native inside a
virtual machine
- and run your IDE and Docker etc inside that VM.
Review comment:
You would need to restore these lines as Hadoop 2.10.x is now buildable
using `Dockerfile` on Virtualbox (as of this commit).
##########
File path: start-build-env.sh
##########
@@ -1,4 +1,4 @@
-#!/bin/bash
Review comment:
It seems like you've copied this file from trunk. Could you please
explain why the changes to this file are needed?
##########
File path: dev-support/docker/Dockerfile
##########
@@ -248,4 +248,4 @@ RUN curl -L -o hugo.deb
https://github.com/gohugoio/hugo/releases/download/v0.30
COPY hadoop_env_checks.sh /root/hadoop_env_checks.sh
RUN chmod 755 /root/hadoop_env_checks.sh
# hadolint ignore=SC2016
-RUN echo '${HOME}/hadoop_env_checks.sh' >> /root/.bashrc
+RUN echo '${HOME}/hadoop_env_checks.sh' >> /root/.bashrc
Review comment:
You shouldn't need to modify this file at all.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
