RFR: 8200626: Restore history for g1ConcurrentMarkThread.*

Erik Helin erik.helin at oracle.com
Wed Apr 4 09:58:40 UTC 2018


On 04/04/2018 11:47 AM, Stefan Johansson wrote:
> 
> 
> On 2018-04-04 11:44, Erik Helin wrote:
>> On 04/04/2018 11:12 AM, Stefan Johansson wrote:
>>>
>>>
>>> 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 and Aleksey, and nice suggestion on pushing quick, 
>> unfortunately I was a bit too slow :)
>>
>> So, here we go again, I uploaded a new bundle, 8200626.01.bundle. To 
>> review, just:
>>
>> wget http://cr.openjdk.java.net/~ehelin/8200626/8200626.01.bundle
>> hg pull -u 8200626.01.bundle
>> hg diff -r 334cdf15428b -r 55f8f5635ef7 # should show no changes
>> hg diff -r 8c78b974cd55 -r 0ed1370f52bb # should show no changes
>>
> Push it!

Just did :) Thanks for helping out with reviewing (and re-reviewing) 
quickly!
Erik

> StefanJ
> 
>> Thanks,
>> Erik
>>
>>> Thanks,
>>> Stefan
>>>
>>>> Thanks,
>>>> -Aleksey
>>>>


More information about the hotspot-gc-dev mailing list