Skip to content

parallelism: Always run one round of the stress test#4484

Closed
dnadlinger wants to merge 2 commits intodlang:masterfrom
dnadlinger:parallelism-test
Closed

parallelism: Always run one round of the stress test#4484
dnadlinger wants to merge 2 commits intodlang:masterfrom
dnadlinger:parallelism-test

Conversation

@dnadlinger
Copy link
Contributor

@dnadlinger dnadlinger commented Jun 27, 2016

This is just to see whether there are any remaining issues on the auto tester.

This is to ensure it does not fall into disrepair. The std{out, err}
output should really be removed, though.
@JackStouffer
Copy link
Contributor

Do you know about this PR: #4399?

No need to duplicate work.

@dnadlinger
Copy link
Contributor Author

That PR indeed was the reason why I just whipped up these changes, but calling them "work" is almost an overstatement – the most time-consuming step was just waiting for the commits to make it to GitHub over my dodgy internet connection.

Note that I didn't bother to deal with the console output, since this is just to see whether the auto tester uncovers any other issues than isNaN being a template now. I'm hoping this will be enough of a base for @burner to work from.

@burner
Copy link
Member

burner commented Jun 28, 2016

@klickverbot it will be, thank you

@DmitryOlshansky
Copy link
Member

LGTM

@dnadlinger
Copy link
Contributor Author

@burner: Do you want to take this over (I'd very much appreciate that, as I can't really spend any more time on it)? Feel free to close in favour of #4399.

@burner
Copy link
Member

burner commented Jul 3, 2016

@klickverbot yes

@burner burner closed this Jul 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants