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 17:11:50 UTC

[GitHub] [arrow] nealrichardson opened a new pull request, #13085: MINOR: [R] Don't use warning() in .onLoad()

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

   According to the [docs](https://stat.ethz.ch/R-manual/R-devel/library/base/html/ns-hooks.html), `packageStartupMessage()` is preferred. With `warning()`, it can show up in the `R CMD INSTALL` output like this:
   
   ```
   installing to D:/a/_temp/Library/00LOCK-arrow/00new/arrow/libs/x64
   ** R
   ** inst
   ** byte-compile and prepare package for lazy loading
   ** help
   *** installing help indices
   ** building package indices
   ** installing vignettes
   ** testing if installed package can be loaded from temporary location
   Warning in fun(libname, pkgname) :
     The tzdb package is not installed. Timezones will not be available.
   ** testing if installed package can be loaded from final location
   Warning in fun(libname, pkgname) :
   ** testing if installed package keeps a record of temporary installation path
     The tzdb package is not installed. Timezones will not be available.
   * MD5 sums
   packaged installation of 'arrow' as arrow_7.0.0.20220505.zip
   * DONE (arrow)
   ```
   
   I will cherry-pick when preparing the 8.0.0 CRAN submission


-- 
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 #13085: MINOR: [R] Don't use warning() in .onLoad()

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

   Benchmark runs are scheduled for baseline = 2add697892b2c0e189c6457e76a2ee2318f37e2c and contender = 1aa4dd67717eaf5cb6cadbd460745e64544214f4. 1aa4dd67717eaf5cb6cadbd460745e64544214f4 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/92cff6ad9cba475cbc326bf674f2123d...ed4b19aee058428dbdc70d07cdb5ad5f/)
   [Finished :arrow_down:0.62% :arrow_up:0.12%] [test-mac-arm](https://conbench.ursa.dev/compare/runs/3703853660b646358c8ae19b4846d9f3...d1db4252e6bd4bfd84bc7ba26efaa2d8/)
   [Finished :arrow_down:1.07% :arrow_up:0.0%] [ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/617892e9ed824e61bcd5b19117a6ceb5...8c181c309cfa43b6b3eaefdef0b41085/)
   [Finished :arrow_down:0.43% :arrow_up:0.0%] [ursa-thinkcentre-m75q](https://conbench.ursa.dev/compare/runs/8458c5d6f01c4a4d8106e2d7ed1e576b...1814728835d54dc5894cad06ae24c5a1/)
   Buildkite builds:
   [Finished] [`1aa4dd67` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/711)
   [Finished] [`1aa4dd67` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/708)
   [Finished] [`1aa4dd67` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/697)
   [Finished] [`1aa4dd67` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/713)
   [Finished] [`2add6978` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/710)
   [Finished] [`2add6978` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/707)
   [Finished] [`2add6978` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/696)
   [Finished] [`2add6978` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/712)
   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] nealrichardson closed pull request #13085: MINOR: [R] Don't use warning() in .onLoad()

Posted by GitBox <gi...@apache.org>.
nealrichardson closed pull request #13085: MINOR: [R] Don't use warning() in .onLoad()
URL: https://github.com/apache/arrow/pull/13085


-- 
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