Skip to content

Conversation

@drin
Copy link
Contributor

@drin drin commented Sep 13, 2022

the IR consumer has been removed, and substrait is being supported instead of compute IR, so the documentation page for IR should be removed to avoid confusion

the IR consumer has been removed, and substrait is being supported
instead of compute IR, so the documentation page for IR should be
removed to avoid confusion
@drin drin changed the title ARROW-17716: Remove IR documentation page ARROW-17716: [Docs] Remove IR documentation page Sep 13, 2022
@github-actions
Copy link

Copy link
Contributor

@cyb70289 cyb70289 left a comment

Choose a reason for hiding this comment

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

+1
cc @westonpace

@pitrou pitrou merged commit 03cf0dd into apache:master Sep 14, 2022
@ursabot
Copy link

ursabot commented Sep 15, 2022

Benchmark runs are scheduled for baseline = 9d33df1 and contender = 03cf0dd. 03cf0dd is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️25.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.78% ⬆️0.17%] test-mac-arm
[Failed ⬇️0.28% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.21% ⬆️0.0%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] 03cf0dd3 ec2-t3-xlarge-us-east-2
[Finished] 03cf0dd3 test-mac-arm
[Failed] 03cf0dd3 ursa-i9-9960x
[Finished] 03cf0dd3 ursa-thinkcentre-m75q
[Finished] 9d33df19 ec2-t3-xlarge-us-east-2
[Finished] 9d33df19 test-mac-arm
[Failed] 9d33df19 ursa-i9-9960x
[Finished] 9d33df19 ursa-thinkcentre-m75q
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

@drin drin deleted the ARROW-17716-remove-ir-documentation branch September 15, 2022 20:31
zagto pushed a commit to zagto/arrow that referenced this pull request Oct 7, 2022
the IR consumer has been removed, and substrait is being supported instead of compute IR, so the documentation page for IR should be removed to avoid confusion

Authored-by: Aldrin M <octalene.dev@pm.me>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants