Fix ensureFreshScopeAfter always cloning scope

[smarter]

PR on top of #3216, only the last commit is new.

[smarter]

test performance please

[dottybot]

performance test scheduled: 1 job(s) in queue, 0 running.

[smarter]

Oops, I should have tested commit N-1 and N to get useful information. @liufengyun is there a way to cancel a running test?

[liufengyun]

You can issue another test for N - 1: test performance please: 196c23c --- the results will accumulate.

[smarter]

Yes but they won't be in the correct order which will look confusing :).

[liufengyun]

Yes but they won't be in the correct order which will look confusing

The hash will help disambiguate :)

I manually cancelled the job, you can reissue the command. I'll add a command to cancel running jobs.

[smarter]

Thanks!

[smarter]

test performance please: 196c23c ea1400f

[dottybot]

performance test scheduled for 196c23c ea1400f: 1 job(s) in queue, 0 running.

[dottybot]

performance test failed:

[check /data/workspace/bench/logs/pull-3222-10-01-16.08.out for more information]

[smarter]

@liufengyun help 😄 ^

[liufengyun]

test performance please: 196c23c ea1400f

There's some issue with the lock of jmh when force killing test.

[dottybot]

performance test scheduled for 196c23c ea1400frnrnheres some issue with the lock of jmh when force killing test: 1 job(s) in queue, 0 running.

[smarter]

ouch, bad parsing :P

[liufengyun]

test performance please: 196c23c ea1400f

[dottybot]

performance test scheduled for 196c23c ea1400frn: 1 job(s) in queue, 0 running.

[liufengyun]

There's some regression with the multi-commits test -- I just manually scheduled the test.

[liufengyun]

The multi-commit command should work now with the fix (already deployed on server): liufengyun/bench@259f626

[smarter]

@liufengyun Are the tests still running?

[liufengyun]

I just checked, there's now some issue with publishing website. I manually published below:

http://dotty-bench.epfl.ch/3222-1/index.html

But it only tests the first commit.

[liufengyun]

test performance please

[dottybot]

performance test scheduled: 1 job(s) in queue, 1 running.

[smarter]

Hmm this is going to be misleading because more stuff has been merged between the last test command 4 hours ago and the new test command now.

[liufengyun]

After the current test finishes, it's possible to run retest performance please to test against the latest master in a new test session.

[dottybot]

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/3222-1/ to see the changes.

Benchmarks is based on merging with master (b7be521)