Skip to content

Conversation

@shivaram
Copy link
Contributor

What changes were proposed in this pull request?

This PR sets the R package version while tagging releases. Note that since R doesn't accept -SNAPSHOT in version number field, we remove that while setting the next version

How was this patch tested?

Tested manually by running locally

@shivaram
Copy link
Contributor Author

cc @rxin

@shivaram shivaram changed the title Set R package version number along with mvn [SPARKR] Set R package version number along with mvn Sep 23, 2016
Also use different tmp extensions for sed to help debugging
@shivaram shivaram changed the title [SPARKR] Set R package version number along with mvn [SPARKR][SPARK-17651] Set R package version number along with mvn Sep 23, 2016
@rxin
Copy link
Contributor

rxin commented Sep 23, 2016

I'm going to merge this since the none of the changes are validated by Jenkins. I will test this by cutting a new RC.

asfgit pushed a commit that referenced this pull request Sep 23, 2016
This PR sets the R package version while tagging releases. Note that since R doesn't accept `-SNAPSHOT` in version number field, we remove that while setting the next version

Tested manually by running locally

Author: Shivaram Venkataraman <[email protected]>

Closes #15223 from shivaram/sparkr-version-change.

(cherry picked from commit 7c38252)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in 7c38252 Sep 23, 2016
@SparkQA
Copy link

SparkQA commented Sep 23, 2016

Test build #65844 has finished for PR 15223 at commit 742a787.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 23, 2016

Test build #65849 has finished for PR 15223 at commit a0122f0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

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