smengcl edited a comment on issue #1218: HDDS-1891. Ozone fs shell command should work with default port when port number is not specified URL: https://github.com/apache/hadoop/pull/1218#issuecomment-519737497 > > One more comment, I think we should make a similar change in BasicOzoneFileSystem.java. > > See, can we have a some utility method, which can be used across 2 classes. > > OzoneFileSystem extends BasicOzoneFileSystem, so this has been taken care. Just test it out locally. While I'm testing it in a docker-compose cluster, it seems that the change of `authority` [here](https://github.com/apache/hadoop/pull/1218/commits/ab93f4bc3fd8d2acf31dac5bbf79c49546eccacd#diff-e48c4ce6b86d4a33c3f38ba8d6d06ea3R129) might have broken something. It throws error: ``` bash-4.2$ ozone fs -ls o3fs://bucket.volume.om/ -ls: Wrong FS: o3fs://bucket.volume.om/, expected: o3fs://bucket.volume.om:9862 ... ``` After I commented out this line, compiled and re-tested, it works again: ``` bash-4.2$ ozone fs -ls o3fs://bucket.volume.om/ Found 1 items -rw-rw-rw- 1 hadoop hadoop 1485 1970-01-01 00:46 o3fs://bucket.volume.om/README.txt ``` FYI my test prep steps: ```bash mvn clean install -f pom.ozone.xml -DskipTests=true -Dmaven.javadoc.skip=true cd hadoop-ozone/dist/target/ozone-0.5.0-SNAPSHOT/compose/ozone docker-compose up -d --scale datanode=3 docker-compose exec om /bin/bash ozone sh volume create /volume ozone sh bucket create /volume/bucket vi /etc/hadoop/core-site.xml # Add fs.o3fs.impl and fs.defaultFS config and save # Ref: https://hadoop.apache.org/ozone/docs/0.4.0-alpha/ozonefs.html ozone fs -put README.txt o3fs:/// ozone fs -ls / ``` Investigating.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
