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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
