RFR: JDK-8298597 : HdrSeq: support for a merge() method [v3]

Y. Srinivas Ramakrishna ysr at openjdk.org
Mon Dec 19 18:35:28 UTC 2022


On Mon, 19 Dec 2022 17:53:34 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> Y. Srinivas Ramakrishna has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
>> 
>>  - Merge branch 'master' into stats_merge
>>  - Merge branch 'master' into stats_merge
>>  - Safety tests for decayed stats, until implemented.
>>  - gtest for merge.
>>  - Vanilla merge test for ShenandoahNumberSeq; needs to be extended some.
>>  - Changes based on experience with uses in RS scan stats.
>>    Fixed some bugs.
>>    
>>    -- We still need to implement a few vanilla tests for the merge method.
>>    -- Planning to defer the work on decayed stats (which will be delivered
>>    separately in a lower-priority sibling ticket)
>>  - Merge branch 'master' into stats_merge
>>  - More merge() implementation.
>>    -- Need to think about merge of decaying stats in AbsSeq.
>>    -- Need to add tests.
>>  - Interim checkin of code w/beginnings of merge() support. Some
>>    implementations are still stubbed out and need to be written.
>>  - First cut at merge. More changes to come. May not build yet.
>
> Marked as reviewed by kdnilsen (Committer).

Pending sponsorship; thanks for the review @kdnilsen!

-------------

PR: https://git.openjdk.org/shenandoah/pull/184


More information about the shenandoah-dev mailing list