RFR: JDK-8199729 - Usage Logger open sourcing

[Sharath Ballal]

Hello All,

 

Based on internal discussions, we have decided to put this open sourcing activity on hold.

 

Hence I would like to withdraw the review request at this time. Thanks for your time and feedback. Apologies for any inconvenience this might have caused.

 

 

Thanks,

Sharath

 

 

From: Sharath Ballal 
Sent: Thursday, May 31, 2018 3:22 PM
To: Bernd Eckenfels; serviceability-dev
Subject: RE: RFR: JDK-8199729 - Usage Logger open sourcing

 

Thanks for the comments Bernd.  I will work on them and send a webrev once done.

 

Thanks,

Sharath

 

 

From: Bernd Eckenfels [mailto:ecki at zusammenkunft.net] 
Sent: Wednesday, May 30, 2018 10:54 PM
To: serviceability-dev
Subject: Re: RFR: JDK-8199729 - Usage Logger open sourcing

 

Would it be possible to remove all the duplicate configuration code from the PostVMInitHook and just call run() { UsageLoggerClient.logJavaUsage(); }? This way all config code can be in one place.

 

I also wonder if the ORCL_ prefix can and should be removed from all constants?

 

Beside that I would return getAbsoluteFile() from the priveledged sections.

 

Gruß

Bernd 

 

-- 

https://Bernd.eckenfels.net

 

On Wed, May 30, 2018 at 6:21 PM +0200, "Sharath Ballal" <HYPERLINK "mailto:sharath.ballal at oracle.com"sharath.ballal at oracle.com> wrote:

Hello,

 

Requesting reviews for Usage Logger open sourcing (earlier known as Usage Tracker).

 

ID: https://bugs.openjdk.java.net/browse/JDK-8199729 

Webrev: http://cr.openjdk.java.net/~sballal/8199729/webrev.00/ 

 

Mach5 run has been successful for Usage Logger tests and hs-tier1, hs-tier2.

 

Thanks,

Sharath

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180531/25ca22de/attachment.html>