JDK-8298597 : HdrSeq: support for a merge() method
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2022-12-12
  • Updated: 2025-05-12
  • Resolved: 2022-12-19
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
tbdFixed
Related Reports
Cloners :  
Relates :  
Relates :  
Relates :  
Description
For GenShen's RS scan instrumentation, we want support for a merge() method for HdrSeq:s.

The rough shape of the spec for the proposed merge() method in HdrSeq class is:

// Merge the data in this HdrSeq into the HdrSeq, seq2, in the argument
// to the method, optionally clearing the data in this HdrSeq.
void merge(HdrSeq* seq2);

This will likely necessitate having the same or similar methods in
superclass hierarchy.

A PR will be opened once the changes have been implemented, refined,
tested and ready to initial discussion. Opening this issue in anticipation.
Comments
A pull request was submitted for review. URL: https://git.openjdk.org/shenandoah/pull/184 Date: 2022-12-15 19:33:36 +0000
16-12-2022

Splitting off the handling of decaying statistics into a separate ticket, deferring that work for now.
16-12-2022

The issue of updating the decaying statistics via a merge method requires careful thought. As it stands today, decaying statistics are not invoked or used in the case of HdrSeq:s subject to merge in the near-future. This space will be updated with a suitable design and possible API adjustments when we have a proposal ready.
14-12-2022