[GitHub] [ozone] pratapchandu commented on a diff in pull request #3795: HDDS-6664. Implements getUri in TrashOzoneFileSystem

[GitBox <gi...@apache.org>]

pratapchandu commented on code in PR #3795:
URL: https://github.com/apache/ozone/pull/3795#discussion_r988661053


##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/TrashOzoneFileSystem.java:
##########
@@ -123,8 +126,15 @@ private void submitRequest(OzoneManagerProtocolProtos.OMRequest omRequest)
 
   @Override
   public URI getUri() {
-    throw new UnsupportedOperationException(
-        "fs.getUri() not implemented in TrashOzoneFileSystem");
+    URI uri = null;
+    try {
+      uri = new URIBuilder().setScheme(OZONE_URI_SCHEME)
+              .setHost("ofs.trash")

Review Comment:
   Defined OZONE_O3TRASH_URI_SCHEME and used it. Without setHost() , it throws a URISyntaxException. 



-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org