You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by GitBox <gi...@apache.org> on 2022/12/07 11:40:35 UTC
[GitHub] [ambari] timyuer commented on a diff in pull request #3590: AMBARI-25789: Support BIGTOP-3.2.0 stack upgrade
timyuer commented on code in PR #3590:
URL: https://github.com/apache/ambari/pull/3590#discussion_r1042101305
##########
ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java:
##########
@@ -78,18 +78,23 @@ public UpgradeCheckResult perform(UpgradeCheckRequest request) throws AmbariExce
final StackId targetStackId = new StackId(repositoryVersion.getStackId());
- if (!repositoryVersion.getVersion().matches("^\\d+(\\.\\d+)*\\-\\d+$")) {
- String message = MessageFormat.format(
- "The Repository Version {0} for Stack {1} must contain a \"-\" followed by a build number. "
- + "Make sure that another registered repository does not have the same repo URL or "
- + "shares the same build number. Next, try reinstalling the Repository Version.",
- repositoryVersion.getVersion(), targetStackId.getStackVersion());
-
- result.getFailedOn().add("Repository Version " + repositoryVersion.getVersion());
- result.setStatus(UpgradeCheckStatus.FAIL);
- result.setFailReason(message);
- return result;
- }
+ /**
+ * Because the version for BigTop is BIGTOP-3.2.0,
+ * But the version for HDP is HDP-x.x.x.x-x,
+ * So to be compatible with both HDP and BIGTOP, comment out this.
+ */
+ // if (!repositoryVersion.getVersion().matches("^\\d+(\\.\\d+)*\\-\\d+$")) {
+ // String message = MessageFormat.format(
+ // "The Repository Version {0} for Stack {1} must contain a \"-\" followed by a build number. "
+ // + "Make sure that another registered repository does not have the same repo URL or "
+ // + "shares the same build number. Next, try reinstalling the Repository Version.",
+ // repositoryVersion.getVersion(), targetStackId.getStackVersion());
+
+ // result.getFailedOn().add("Repository Version " + repositoryVersion.getVersion());
+ // result.setStatus(UpgradeCheckStatus.FAIL);
+ // result.setFailReason(message);
+ // return result;
+ // }
Review Comment:
Update as comment.
--
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: dev-unsubscribe@ambari.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ambari.apache.org
For additional commands, e-mail: dev-help@ambari.apache.org