Skip to content

JDBC version change for 2.2 release#3946

Merged
sougou merged 1 commit intovitessio:masterfrom
harshit-gangal:java-version-2.2
May 18, 2018
Merged

JDBC version change for 2.2 release#3946
sougou merged 1 commit intovitessio:masterfrom
harshit-gangal:java-version-2.2

Conversation

@harshit-gangal
Copy link
Copy Markdown
Member

Signed-off-by: Harshit Gangal harshit.gangal@gmail.com

Signed-off-by: Harshit Gangal <harshit.gangal@gmail.com>
@harshit-gangal harshit-gangal requested a review from sougou May 18, 2018 20:49
Copy link
Copy Markdown
Contributor

@sougou sougou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hah. we're donwgrading :)

@sougou sougou merged commit 66e84fa into vitessio:master May 18, 2018
@michael-berlin
Copy link
Copy Markdown
Contributor

btw: The -SNAPSHOT version in the pom.xml on the master branch is not relevant for the release.

That's because as part of https://vitess.io/internal/release-instructions/#java-packages you need to change (and commit) it on the release branch anyway.

Then, after the release you point it to the next planned version, see: https://vitess.io/internal/release-instructions/#bump-java-snapshot-version

In this case, the next version is 3.0, so the pom.xml files could have just stayed as they are in the master branch.

@harshit-gangal
Copy link
Copy Markdown
Member Author

Yes. I will bump it to 3.0 after release. Just so that 2.2 is not yet released and I wanted to change the Java Driver version to 2.2, changed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants