updated the elasticsearch versioning format

Moving to from `X.Y.Z.beta1`/`X.Y.Z.RC1` to `X.Y.Z-beta1`/`X.Y.Z-rc1`
This commit is contained in:
uboness 2015-07-13 19:45:53 +02:00
parent 17906ca7d6
commit b40186652c
18 changed files with 42 additions and 23 deletions

View File

@ -6,7 +6,7 @@
<parent> <parent>
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>
<artifactId>elasticsearch-parent</artifactId> <artifactId>elasticsearch-parent</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>

View File

@ -499,7 +499,7 @@ public class Version {
if (snapshot = version.endsWith("-SNAPSHOT")) { if (snapshot = version.endsWith("-SNAPSHOT")) {
version = version.substring(0, version.length() - 9); version = version.substring(0, version.length() - 9);
} }
String[] parts = version.split("\\."); String[] parts = version.split("\\.|\\-");
if (parts.length < 3 || parts.length > 4) { if (parts.length < 3 || parts.length > 4) {
throw new IllegalArgumentException("the version needs to contain major, minor and revision, and optionally the build: " + version); throw new IllegalArgumentException("the version needs to contain major, minor and revision, and optionally the build: " + version);
} }
@ -518,7 +518,7 @@ public class Version {
if (buildStr.startsWith("Beta") || buildStr.startsWith("beta")) { if (buildStr.startsWith("Beta") || buildStr.startsWith("beta")) {
build = Integer.parseInt(buildStr.substring(4)); build = Integer.parseInt(buildStr.substring(4));
} }
if (buildStr.startsWith("RC")) { if (buildStr.startsWith("RC") || buildStr.startsWith("rc")) {
build = Integer.parseInt(buildStr.substring(2)) + 50; build = Integer.parseInt(buildStr.substring(2)) + 50;
} }
} }
@ -591,13 +591,18 @@ public class Version {
sb.append(major).append('.').append(minor).append('.').append(revision); sb.append(major).append('.').append(minor).append('.').append(revision);
if (isBeta()) { if (isBeta()) {
if (major >= 2) { if (major >= 2) {
sb.append(".beta"); sb.append("-beta");
} else { } else {
sb.append(".Beta"); sb.append(".Beta");
} }
sb.append(build); sb.append(build);
} else if (build < 99) { } else if (build < 99) {
sb.append(".RC").append(build - 50); if (major >= 2) {
sb.append("-rc");
} else {
sb.append(".RC");
}
sb.append(build - 50);
} }
return sb.toString(); return sb.toString();
} }
@ -644,6 +649,10 @@ public class Version {
return build < 50; return build < 50;
} }
public boolean isRC() {
return build > 50 && build < 99;
}
public static class Module extends AbstractModule { public static class Module extends AbstractModule {
private final Version version; private final Version version;

View File

@ -138,7 +138,7 @@ public class VersionTests extends ElasticsearchTestCase {
public void testToString() { public void testToString() {
// with 2.0.beta we lowercase // with 2.0.beta we lowercase
assertEquals("2.0.0.beta1", Version.V_2_0_0_beta1.number()); assertEquals("2.0.0-beta1", Version.V_2_0_0_beta1.number());
assertEquals("1.4.0.Beta1", Version.V_1_4_0_Beta1.number()); assertEquals("1.4.0.Beta1", Version.V_1_4_0_Beta1.number());
assertEquals("1.4.0", Version.V_1_4_0.number()); assertEquals("1.4.0", Version.V_1_4_0.number());
} }
@ -189,7 +189,17 @@ public class VersionTests extends ElasticsearchTestCase {
logger.info("Checking " + v); logger.info("Checking " + v);
assertEquals("Version id " + field.getName() + " does not point to " + constantName, v, Version.fromId(versionId)); assertEquals("Version id " + field.getName() + " does not point to " + constantName, v, Version.fromId(versionId));
assertEquals("Version " + constantName + " does not have correct id", versionId, v.id); assertEquals("Version " + constantName + " does not have correct id", versionId, v.id);
assertEquals("V_" + v.number().replace('.', '_'), constantName); if (v.major >= 2) {
String number = v.number();
if (v.isBeta()) {
number = number.replace("-beta", "_beta");
} else if (v.isRC()) {
number = number.replace("-rc", "_rc");
}
assertEquals("V_" + number.replace('.', '_'), constantName);
} else {
assertEquals("V_" + v.number().replace('.', '_'), constantName);
}
// only the latest version for a branch should be a snapshot (ie unreleased) // only the latest version for a branch should be a snapshot (ie unreleased)
String branchName = "" + v.major + "." + v.minor; String branchName = "" + v.major + "." + v.minor;

View File

@ -2,7 +2,7 @@
<modelVersion>4.0.0</modelVersion> <modelVersion>4.0.0</modelVersion>
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>
<artifactId>elasticsearch-dev-tools</artifactId> <artifactId>elasticsearch-dev-tools</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
<name>Elasticsearch Build Resources</name> <name>Elasticsearch Build Resources</name>
<parent> <parent>
<groupId>org.sonatype.oss</groupId> <groupId>org.sonatype.oss</groupId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-analysis-icu</artifactId> <artifactId>elasticsearch-analysis-icu</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-analysis-kuromoji</artifactId> <artifactId>elasticsearch-analysis-kuromoji</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-analysis-phonetic</artifactId> <artifactId>elasticsearch-analysis-phonetic</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-analysis-smartcn</artifactId> <artifactId>elasticsearch-analysis-smartcn</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-analysis-stempel</artifactId> <artifactId>elasticsearch-analysis-stempel</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-cloud-aws</artifactId> <artifactId>elasticsearch-cloud-aws</artifactId>

View File

@ -18,7 +18,7 @@ governing permissions and limitations under the License. -->
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-cloud-azure</artifactId> <artifactId>elasticsearch-cloud-azure</artifactId>

View File

@ -18,7 +18,7 @@ governing permissions and limitations under the License. -->
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-cloud-gce</artifactId> <artifactId>elasticsearch-cloud-gce</artifactId>

View File

@ -18,7 +18,7 @@ governing permissions and limitations under the License. -->
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-delete-by-query</artifactId> <artifactId>elasticsearch-delete-by-query</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-lang-javascript</artifactId> <artifactId>elasticsearch-lang-javascript</artifactId>

View File

@ -7,7 +7,7 @@
<parent> <parent>
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<artifactId>elasticsearch-lang-python</artifactId> <artifactId>elasticsearch-lang-python</artifactId>

View File

@ -7,7 +7,7 @@
<groupId>org.elasticsearch.plugin</groupId> <groupId>org.elasticsearch.plugin</groupId>
<artifactId>elasticsearch-plugin</artifactId> <artifactId>elasticsearch-plugin</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
<packaging>pom</packaging> <packaging>pom</packaging>
<name>Elasticsearch Plugin POM</name> <name>Elasticsearch Plugin POM</name>
<inceptionYear>2009</inceptionYear> <inceptionYear>2009</inceptionYear>
@ -15,7 +15,7 @@
<parent> <parent>
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>
<artifactId>elasticsearch-parent</artifactId> <artifactId>elasticsearch-parent</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
</parent> </parent>
<properties> <properties>

View File

@ -6,7 +6,7 @@
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>
<artifactId>elasticsearch-parent</artifactId> <artifactId>elasticsearch-parent</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
<packaging>pom</packaging> <packaging>pom</packaging>
<name>Elasticsearch Parent POM</name> <name>Elasticsearch Parent POM</name>
<description>Elasticsearch Parent POM</description> <description>Elasticsearch Parent POM</description>

View File

@ -2,7 +2,7 @@
<modelVersion>4.0.0</modelVersion> <modelVersion>4.0.0</modelVersion>
<groupId>org.elasticsearch</groupId> <groupId>org.elasticsearch</groupId>
<artifactId>elasticsearch-rest-api-spec</artifactId> <artifactId>elasticsearch-rest-api-spec</artifactId>
<version>2.0.0.beta1-SNAPSHOT</version> <version>2.0.0-beta1-SNAPSHOT</version>
<name>Elasticsearch Rest API Spec</name> <name>Elasticsearch Rest API Spec</name>
<parent> <parent>
<groupId>org.sonatype.oss</groupId> <groupId>org.sonatype.oss</groupId>