RFR: 8200626: Restore history for g1ConcurrentMarkThread.*

Stefan Johansson stefan.johansson at oracle.com
Wed Apr 4 09:12:46 UTC 2018



On 2018-04-04 11:04, Aleksey Shipilev wrote:
> On 04/04/2018 10:59 AM, Erik Helin wrote:
>>> "hg log -G -l 5" or something would be nice to see too.
>>
>> Since this change involves so much hg metadata that is important, I uploaded a hg bundle:
>>
>> http://cr.openjdk.java.net/~ehelin/8200626/8200626.bundle
>>
>> To review the change, just:
>>
>> $ # download the bundle
>> $ wget http://cr.openjdk.java.net/~ehelin/8200626/8200626.bundle
>>
>> $ hg incoming 8200626.bundle # list the changesets in the bundle
>> $ hg pull -u 8200626.bundle # pull the changesets into your repository
>> $ hg log --graph # inspect the log
>>
>> $ hg diff -r be49a19be03b:0ab2411f270d # verify that merge is benign
>> $ hg diff -r 8c78b974cd55 -r 0ed1370f52bb # verify my change is
>>                                            # identical to Leo's
> 
> Yes, verified. Thanks!
> 
> I think it is trivial code-wise, and the risk of waiting 24 hrs opens up the opportunity to screw up
> the re-merge later. Therefore, I think we should push it right away. Maybe another Reviewer should
> double-check :)
> 
I also verified it. Good work. I agree with Aleksey, push it now.

Thanks,
Stefan

> Thanks,
> -Aleksey
> 



More information about the hotspot-gc-dev mailing list