datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard

Automate postrelease publishing to Homebrew

Open iajoiner opened this issue 1 year ago • 3 comments

Which issue does this PR close?

Closes #3506.

Rationale for this change

What changes are included in this PR?

dev/release/publish_homebrew.sh

Are there any user-facing changes?

No

iajoiner avatar Sep 16 '22 01:09 iajoiner

@andygrove @alamb This will be in draft until DataFusion 13.0.0 gets released because we will test it then. I will also change the documentation to reflect the change.

iajoiner avatar Sep 16 '22 01:09 iajoiner

Codecov Report

Merging #3507 (cc3893b) into master (3319220) will increase coverage by 0.00%. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3507   +/-   ##
=======================================
  Coverage   85.75%   85.75%           
=======================================
  Files         299      299           
  Lines       55311    55311           
=======================================
+ Hits        47431    47433    +2     
+ Misses       7880     7878    -2     
Impacted Files Coverage Δ
datafusion/expr/src/logical_plan/plan.rs 77.68% <0.00%> (ø)
datafusion/core/src/physical_plan/metrics/value.rs 87.43% <0.00%> (+0.50%) :arrow_up:
datafusion/expr/src/window_frame.rs 93.27% <0.00%> (+0.84%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Sep 16 '22 02:09 codecov-commenter

Thanks @iajoiner. LGTM. Could you update the release instructions (dev/release/README.md) as well as part of this PR? Once that is done we should go ahead and merge this. If we hit any issues when releasing 13.0.0 we can PR any fixes needed.

andygrove avatar Sep 19 '22 15:09 andygrove

@andygrove Sure! I will do that today.

iajoiner avatar Sep 29 '22 16:09 iajoiner

@andygrove Here we go!

iajoiner avatar Sep 29 '22 19:09 iajoiner

@andygrove Sure! Done!

iajoiner avatar Oct 03 '22 16:10 iajoiner

Benchmark runs are scheduled for baseline = 6b282ec9585a1ca150c5760ba0536876706c6d35 and contender = 010b3526a763b5eca73bba501965901d605935d6. 010b3526a763b5eca73bba501965901d605935d6 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. Conbench compare runs links: [Skipped :warning: Benchmarking of arrow-datafusion-commits is not supported on ec2-t3-xlarge-us-east-2] ec2-t3-xlarge-us-east-2 [Skipped :warning: Benchmarking of arrow-datafusion-commits is not supported on test-mac-arm] test-mac-arm [Skipped :warning: Benchmarking of arrow-datafusion-commits is not supported on ursa-i9-9960x] ursa-i9-9960x [Skipped :warning: Benchmarking of arrow-datafusion-commits is not supported on ursa-thinkcentre-m75q] ursa-thinkcentre-m75q Buildkite builds: 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

ursabot avatar Oct 03 '22 18:10 ursabot