[GitHub] [cloudstack] SadiJr commented on a diff in pull request #6589: [Veeam] disable jobs but keep backups

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

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


##########
utils/src/main/java/com/cloud/utils/UuidUtils.java:
##########
@@ -28,6 +28,10 @@ public final static String first(String uuid) {
         return uuid.substring(0, uuid.indexOf('-'));
     }
 
+    public static boolean isUuid(String data) {

Review Comment:
   From my point of view, using validate in the name of a method gives the impression that if the validation fails, an exception will be thrown. As in this case only true or false is returned, I decided to create a method whose name, again from my point of view, makes more sense.



-- 
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