You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2022/04/04 19:40:04 UTC

[GitHub] [cloudstack] SadiJr commented on a diff in pull request #6185: Fix spelling

SadiJr commented on code in PR #6185:
URL: https://github.com/apache/cloudstack/pull/6185#discussion_r842078684


##########
services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java:
##########
@@ -1139,7 +1139,7 @@ protected File downloadFromUrlToNfs(String url, NfsTO nfs, String path, String n
             }
             return new File(destFile.getAbsolutePath());
         } catch (IOException e) {
-            s_logger.debug("Faild to get url:" + url + ", due to " + e.toString());
+            s_logger.debug("Failed to get url:" + url + ", due to " + e.toString());

Review Comment:
   LGTM, just a small suggestion:
   
   ```suggestion
               s_logger.debug("Failed to get url: " + url + ", due to " + e.toString());
   ```



-- 
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: commits-unsubscribe@cloudstack.apache.org

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