Skip to content

ARROW-18074: [CI] Running ctest for PyArrow C++ not needed anymore #14435

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pitrou merged 1 commit into from
Oct 17, 2022
Merged

ARROW-18074: [CI] Running ctest for PyArrow C++ not needed anymore #14435

pitrou merged 1 commit into from
Oct 17, 2022

Conversation

@AlenkaF
Copy link
Member

@AlenkaF AlenkaF commented Oct 17, 2022

As #14117 is merged separate run of ctest for PyArrow C++ tests is not needed anymore and is removed in this PR.

@github-actions
Copy link

github-actions bot commented Oct 17, 2022

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

@github-actions
Copy link

github-actions bot commented Oct 17, 2022

⚠️ Ticket has not been started in JIRA, please click 'Start Progress'.

pitrou
pitrou approved these changes Oct 17, 2022
View reviewed changes
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

Nice to see the PyArrow build chain progressively getting simpler :-)

@pitrou pitrou merged commit b7e60c7 into apache:master Oct 17, 2022
@AlenkaF AlenkaF deleted the ARROW-17345 branch October 17, 2022 10:35
kou pushed a commit that referenced this pull request Oct 20, 2022
@AlenkaF @kou
ARROW-18074: [CI] Running ctest for PyArrow C++ not needed anymore (#…
5e48f9f 
…14435)

As #14117 is merged separate run of `ctest` for `PyArrow C++` tests is not needed anymore and is removed in this PR.

Authored-by: Alenka Frim <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
@ursabot
Copy link

ursabot commented Oct 27, 2022

Benchmark runs are scheduled for baseline = 66874f5 and contender = b7e60c7. b7e60c7 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Failed ⬇️0.56% ⬆️0.0%] test-mac-arm
[Finished ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Failed] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] b7e60c75 ec2-t3-xlarge-us-east-2
[Failed] b7e60c75 test-mac-arm
[Finished] b7e60c75 ursa-i9-9960x
[Failed] b7e60c75 ursa-thinkcentre-m75q
[Finished] 66874f5a ec2-t3-xlarge-us-east-2
[Failed] 66874f5a test-mac-arm
[Finished] 66874f5a ursa-i9-9960x
[Failed] 66874f5a 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

@ursabot
Copy link

ursabot commented Oct 27, 2022

['Python', 'R'] benchmarks have high level of regressions.
test-mac-arm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pitrou pitrou pitrou approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AlenkaF @ursabot @pitrou