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.
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
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.
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
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
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
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
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