RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-01-28 Thread Steven Schlansker
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8006627 I have created a patch that dramatically improves UUID to/from string performance. Please find below a webrev with my proposed changes. Thanks in advance for any feedback on the contents. I do not believe I have a committer lined up yet.

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-01-30 Thread Mike Duigou
Good work Steven! Some initial comments; - The changes to Long should be in a separate issue. It's generally encouraged for changesets to focus on only one change. Sorry, yes, it's additional overhead. - I would like to see if performed of toString() can be improved further by using String(c

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-01 Thread Steven Schlansker
On Jan 30, 2013, at 11:42 AM, Mike Duigou wrote: > Good work Steven! Thanks for the review :) > > Some initial comments; > > - The changes to Long should be in a separate issue. It's generally > encouraged for changesets to focus on only one change. Sorry, yes, it's > additional overhead.

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-01 Thread Mike Duigou
On Feb 1 2013, at 10:13 , Steven Schlansker wrote: > > On Jan 30, 2013, at 11:42 AM, Mike Duigou wrote: > >> Good work Steven! > > Thanks for the review :) >> >> Some initial comments; >> >> - The changes to Long should be in a separate issue. It's generally >> encouraged for changesets t

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-03 Thread Steven Schlansker
On Feb 1, 2013, at 11:42 AM, Mike Duigou wrote: > > I have created another issue 8007398 for the changes to Long. We can even > test and push the two issues at the same time. Separating them into two > changesets makes the intent easier to follow for future maintainers. > > We can use the sam

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-03 Thread Mike Duigou
This looks fairly much complete to me. I will run it through the some testing here and then push your patch. Great work and thank you! Mike On Feb 3 2013, at 21:26 , Steven Schlansker wrote: > > On Feb 1, 2013, at 11:42 AM, Mike Duigou wrote: >> >> I have created another issue 8007398 for t

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-11 Thread Mike Duigou
I have been following up on this issue. I am going to have to adapt the code a bit because there have been some changes in JDK8. I won't forget this issue though. It is possible I may not have time to backport it to Java 7. Mile On Feb 3 2013, at 21:26 , Steven Schlansker wrote: > > On Feb 1

Re: RFR (S) CR 8006627: Improving performance and reducing object allocations of java.util.UUID to/from string

2013-02-11 Thread Steven Schlansker
On Feb 11, 2013, at 9:04 PM, Mike Duigou wrote: > I have been following up on this issue. I am going to have to adapt the code > a bit because there have been some changes in JDK8. > > I won't forget this issue though. It is possible I may not have time to > backport it to Java 7. > Please l