RFR (XS) JDK-8164482,[REDO] G1 does not implement	millis_since_last_gc which is needed by RMI GC
    Thomas Schatzl 
    thomas.schatzl at oracle.com
       
    Mon Sep  5 11:25:36 UTC 2016
    
    
  
Hi,
On Mon, 2016-09-05 at 13:17 +0200, Thomas Schatzl wrote:
> Hi again,
> 
> On Mon, 2016-09-05 at 13:13 +0200, Thomas Schatzl wrote:
> > 
> > Hi,
> > 
> > On Fri, 2016-09-02 at 09:40 -0400, Joseph Provino wrote:
> > > 
> > > 
> > > Please review this small change.
> > > JDK-8164482 [REDO] G1 does not implement millis_since_last_gc
> > > which
> > > is needed by RMI GC
> > > webrev:  http://cr.openjdk.java.net/~jprovino/8164482/webrev.00
> >   looks good.
>   after having another look, I do not think the change is
> good: millis_since_last_gc() tracks the time since any last gc
> (probably collection pause), so the last gc time needs to be updated
> at the end of every gc, not only full gcs.
> 
> The member and the getter name should also reflect this.
  also, what kind of testing did this change go through?
Please always mention this during the first review request email.
Thanks,
  Thomas
    
    
More information about the hotspot-gc-dev
mailing list