adoroszlai commented on a change in pull request #1667: URL: https://github.com/apache/ozone/pull/1667#discussion_r544610169
########## File path: hadoop-ozone/dist/src/main/compose/ozone-mr/hadoop27/docker-config ########## @@ -18,4 +18,7 @@ CORE-SITE.xml_fs.AbstractFileSystem.o3fs.impl=org.apache.hadoop.fs.ozone.OzFs CORE-SITE.xml_fs.AbstractFileSystem.ofs.impl=org.apache.hadoop.fs.ozone.RootedOzFs MAPRED-SITE.XML_mapreduce.application.classpath=/opt/hadoop/share/hadoop/mapreduce/*:/opt/hadoop/share/hadoop/mapreduce/lib/*:/opt/ozone/share/ozone/lib/[email protected]@.jar +HADOOP_CLASSPATH=/opt/ozone/share/ozone/lib/[email protected]@.jar +OZONE_CLASSPATH= Review comment: > setting it to empty on purpose Exactly: without it `HADOOP_CLASSPATH` containing OzoneFS jar would be picked up, which is bad for Ozone's health. Actually, this is the primary motivation for this entire change. From description of HDDS-4525: > severe problem happens if we would like to access Ozone filesystem both via `ozone` and `hadoop` commands. The latter needs shaded Ozone FS JAR in `HADOOP_CLASSPATH`. The same `HADOOP_CLASSPATH` results in `ClassNotFound` for `ozone`. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
