Skip to content

ARROW-14979: [C++] Fix process leaks in GCS integration tests#11852

Closed
coryan wants to merge 1 commit into
apache:masterfrom
coryan:ARROW-14979-gcsfs-tests-leak-processes
Closed

ARROW-14979: [C++] Fix process leaks in GCS integration tests#11852
coryan wants to merge 1 commit into
apache:masterfrom
coryan:ARROW-14979-gcsfs-tests-leak-processes

Conversation

@coryan

@coryan coryan commented Dec 3, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Dec 3, 2021

Copy link
Copy Markdown

@coryan coryan marked this pull request as ready for review December 3, 2021 20:26
@coryan

coryan commented Dec 4, 2021

Copy link
Copy Markdown
Contributor Author

The failure on R / rhub/debian-gcc-devel:latest seems unrelated, please take a look.

@pitrou pitrou changed the title ARROW-14979: [C++] fix process leaks in GCS integration tests ARROW-14979: [C++] Fix process leaks in GCS integration tests Dec 7, 2021

@pitrou pitrou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you @coryan

@pitrou pitrou closed this in a4076cd Dec 7, 2021
@coryan coryan deleted the ARROW-14979-gcsfs-tests-leak-processes branch December 7, 2021 15:20
@ursabot

ursabot commented Dec 7, 2021

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = ed8d415 and contender = a4076cd. a4076cd 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
[Scheduled] ursa-i9-9960x
[Finished ⬇️0.22% ⬆️0.0%] ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python. Runs only benchmarks with cloud = True
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

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.

3 participants