You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/06 22:43:51 UTC

[GitHub] [arrow] kou opened a new pull request, #13089: ARROW-16497: [R] Update version in NEWS.md

kou opened a new pull request, #13089:
URL: https://github.com/apache/arrow/pull/13089

   This solves the following CI failure:
   
   https://github.com/apache/arrow/runs/6329293119?check_suite_focus=true
   
       Failure: test_version_pre_tag(PrepareTest)
       /home/runner/work/arrow/arrow/dev/release/01-prepare-test.rb:226:in `test_version_pre_tag'
            223:     end
            224:
            225:     prepare("VERSION_PRE_TAG")
         => 226:     assert_equal(expected_changes.sort_by {|diff| diff[:path]},
            227:                  parse_patch(git("log", "-n", "1", "-p")))
            228:   end
            229: end
       ?  {:hunks=>[["-# arrow 8.0.0.9000", "+# arrow 9.0.0"]], :path=>"r/NEWS.md"},
       ?                       7


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] kou commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
kou commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120588429

   https://github.com/apache/arrow/pull/13095


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] kou commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
kou commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120567783

   Ah, I got it.
   
   `0,/.../` syntax used in https://github.com/apache/arrow/blob/master/dev/release/utils-prepare.sh#L131-L138 is a GNU sed extension. Sorry.
   
   I'll merge this and fix the problem in a separated pull request to work with sed on macOS.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] kszucs commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
kszucs commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120483094

   I had no errors in neither of the scripts.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] nealrichardson commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
nealrichardson commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120442486

   Don't we bump all of the versions on master after the release? Any idea why this didn't get handled by that?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] ursabot commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
ursabot commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1123017716

   Benchmark runs are scheduled for baseline = 953a6ded05dd5f832d2757b1b4eab51cb35349b6 and contender = 26aad1432922f320d9c1752ae304e34fd5c032a9. 26aad1432922f320d9c1752ae304e34fd5c032a9 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
   Conbench compare runs links:
   [Finished :arrow_down:0.0% :arrow_up:0.0%] [ec2-t3-xlarge-us-east-2](https://conbench.ursa.dev/compare/runs/d43a796faad2407890fd8b325bca0340...761e61a92ec74d03902bbfcf1e058cf7/)
   [Failed :arrow_down:1.32% :arrow_up:0.0%] [test-mac-arm](https://conbench.ursa.dev/compare/runs/acc17d799f9c42ddaf7de6fd8c3f0d34...df9fc888c4734e2798cb0ac5f4c3dd3e/)
   [Finished :arrow_down:0.0% :arrow_up:0.0%] [ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/2fa686366a48432c8492ec286e709cb3...1dfe6383d00b4478aa8779bd07766f20/)
   [Finished :arrow_down:0.2% :arrow_up:0.04%] [ursa-thinkcentre-m75q](https://conbench.ursa.dev/compare/runs/7a880b87b2de46c9b72d06074520c1d8...fc0fb540001f4facbdeaf9e4e38b827b/)
   Buildkite builds:
   [Finished] [`26aad143` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/720)
   [Finished] [`26aad143` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/717)
   [Finished] [`26aad143` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/707)
   [Finished] [`26aad143` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/722)
   [Finished] [`953a6ded` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/719)
   [Failed] [`953a6ded` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/716)
   [Finished] [`953a6ded` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/706)
   [Finished] [`953a6ded` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/721)
   Supported benchmarks:
   ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
   test-mac-arm: Supported benchmark langs: C++, Python, R
   ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
   ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] kou commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
kou commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120482785

   `# arrow 7.0.0.9000` should be updated to `# arrow 8.0.0` by `dev/release/01-prepare.sh` but https://github.com/apache/arrow/commit/c3d031250a7fdcfee5e576833bf6f39097602c30 doesn't include any change for `r/NEWS.md`.
   `# arrow 8.0.0.9000` should be added before the updated `# arrow 8.0.0` by `dev/release/post-12-bump-versions.sh` but https://github.com/apache/arrow/commit/4c21fd12f93e4853c03c05919ffb22c6bb8f09b0 doesn't include any change for `r/NEWS.md`.
   
   @kszucs Did you get any error when you run `dev/release/01-prepare.sh` and `dev/release/post-12-bump-versions.sh`? Or did you get any conflict when you merge these commits?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] github-actions[bot] commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120059800

   :warning: Ticket **has not been started in JIRA**, please click 'Start Progress'.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] github-actions[bot] commented on pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #13089:
URL: https://github.com/apache/arrow/pull/13089#issuecomment-1120059786

   https://issues.apache.org/jira/browse/ARROW-16497


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] kou closed pull request #13089: ARROW-16497: [R] Update version in NEWS.md

Posted by GitBox <gi...@apache.org>.
kou closed pull request #13089: ARROW-16497: [R] Update version in NEWS.md
URL: https://github.com/apache/arrow/pull/13089


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org