Skip to content
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

[do not merge] Use a Rayon thread pool #59647

Closed
wants to merge 1 commit into from
Closed

[do not merge] Use a Rayon thread pool #59647

wants to merge 1 commit into from

Conversation

Zoxc
Copy link
Contributor

@Zoxc Zoxc commented Apr 2, 2019

Checking performance overhead of doing so.

@rust-highfive
Copy link
Collaborator

r? @varkor

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 2, 2019
@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 2, 2019

@bors try

@bors
Copy link
Contributor

bors commented Apr 2, 2019

⌛ Trying commit c47118e with merge fe7212d2dc97c813a33b4cc75ee787703184212d...

@bors
Copy link
Contributor

bors commented Apr 2, 2019

💔 Test failed - checks-travis

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 2, 2019
@Zoxc Zoxc mentioned this pull request Apr 2, 2019
Closed
@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 3, 2019

@bors try

@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 3, 2019

@bors retry try

@bors
Copy link
Contributor

bors commented Apr 3, 2019

⌛ Trying commit c47118e with merge fe344ee07fff37f3a085d4f1e7d3a192d3486ce4...

@bors
Copy link
Contributor

bors commented Apr 3, 2019

💔 Test failed - checks-travis

@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 3, 2019

@bors retry try

@bors
Copy link
Contributor

bors commented Apr 3, 2019

⌛ Trying commit c47118e with merge 07806ad...

bors added a commit that referenced this pull request Apr 3, 2019
@bors
Auto merge of #59647 - Zoxc:test-rayon-tp, r=<try>
07806ad 
[do not merge] Use a Rayon thread pool

Checking performance overhead of doing so.
@bors
Copy link
Contributor

bors commented Apr 3, 2019

☀️ Try build successful - checks-travis
Build commit: 07806ad

@Zoxc
Copy link
Contributor Author

Zoxc commented Apr 3, 2019

@rust-timer build 07806ad

@rust-timer
Copy link
Collaborator

Success: Queued 07806ad with parent 7641873, comparison URL.

@alexcrichton alexcrichton mentioned this pull request Apr 3, 2019
Closed
@rust-timer
Copy link
Collaborator

Finished benchmarking try commit 07806ad

@Zoxc Zoxc closed this Apr 3, 2019
@Zoxc Zoxc deleted the test-rayon-tp branch April 10, 2019 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@varkor varkor

Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Zoxc @rust-highfive @bors @rust-timer @varkor