[performance] Re-arranging maps in config metadata improves startup time

[Aleksey Shipilev]

On 03/21/2012 07:13 PM, mark.reinhold at oracle.com wrote:
> 2012/3/21 7:47 -0700, alan.bateman at oracle.com:
>> On 21/03/2012 09:14, Aleksey Shipilev wrote:
>>> On 03/15/2012 04:03 PM, Aleksey Shipilev wrote:
>>>> This is rather simple and straight-forward optimization:
>>>> http://people.apache.org/~shade/jigsaw/webrev-rearrange-1/
>>>>
>>>> Please take a look. I'd be happy to see this patch landing in Jigsaw
>>>> code (are there any other steps besides preparing webrev?).
>>
>> We can certainly include this patch to help for the short term, but ultimately
>> this code needs to be replaced with a b-tree implementation.
> 
> Sorry, but I'd rather not see this go in.  All of this code will soon be
> replaced with the b-tree implementation.  Changing it now just makes it
> more tedious to rebase the evolving b-tree patch.

It's definitely your call.

However I'd like to point out that "evolving b-tree patch" is still in
limbo. My patch on the other hand, is ready, is simple, is local, and is
helping the problem now. The better is the enemy of the good.

-Aleksey.