Ok, new and improved (actually keeping the gate's updates), here's v3:

http://cr.opensolaris.org/~bpytlik/ips-2672-v3/


Thanks,
Brock

Brock Pytlik wrote:
> Brock Pytlik wrote:
>   
>> http://cr.opensolaris.org/~bpytlik/ips-2672-v1/
>> has the patch.
>>
>> It makes case-insensitive search both possible and the default behavior. 
>> For local search, there's an option ( -I though I'm definitely open to 
>> alternatives -c?) that allows for case sensitive search. For protocol 
>> reasons, remote search is only case insensitive. There is a small (IMO) 
>> performance hit for both local and remote search. Case sensitive search 
>> still takes the same amount of time: .5-.6 seconds but case-insensitive 
>> search takes .7-.8 seconds.
>>
>> Remote search takes the biggest hit, going from .1-.2 seconds to .6-.7 
>> seconds. For now, I think this is still acceptable. The time it takes 
>> will grow with the number of new search tokens introduced. If this 
>> becomes a performance issue in the future, there are several possible 
>> optimizations that can be made on the server side. My choice would be to 
>> essentially pre-build the case sensitive dictionary in addition to the 
>> case-sensitive dictionary.
>>
>> Thanks,
>> Brock
>>
>>
>> _______________________________________________
>> pkg-discuss mailing list
>> [email protected]
>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
>>   
>>     
> Bah, ignore that v2 webrev, just discovered that automerge apparently 
> choose to blow away a large number of gate changes.
> I'll fix it up and send out v3 in a few mins.
>
>
> Brock
> _______________________________________________
> pkg-discuss mailing list
> [email protected]
> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
>   

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to