ZhendongBai commented on a change in pull request #3349:
URL: https://github.com/apache/hadoop/pull/3349#discussion_r707835411
##########
File path: dev-support/docker/Dockerfile_centos7
##########
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Review comment:
@GauthamBanasandra I rename Dockerfile_centos7 to Dockerfile_centos_7 to
keep consistent with the filename in trunk. and `mvn clean package
-Dhttps.protocols=TLSv1.2 -DskipTests -Pnative,dist -Drequire.fuse
-Drequire.openssl -Drequire.snappy -Drequire.valgrind -Drequire.zstd
-Drequire.test.libhadoop -Pyarn-ui -Dtar -Dmaven.javadoc.skip=true > build.log
2>&1` logs here:
[build.log](https://github.com/apache/hadoop/files/7158247/build.log), and
because some javadocs are illegal, and javadocs check failed, I add
`-Dmaven.javadoc.skip=true` to build command. please review again, thanks.
--
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]
