[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rpm-devel
Subject:    Re: [CVS] RPM: rpm-5_3: rpm/rpmdb/ rpmdb.c
From:       Jeff Johnson <n3npq () mac ! com>
Date:       2011-05-27 15:18:35
Message-ID: FE1668DD-3AF5-41F3-A4A8-7770E1CDED1F () mac ! com
[Download RAW message or body]

On May 27, 2011, at 11:05 AM, Per yvind Karlsen wrote:

> 2011/5/27 Jeff Johnson <n3npq@mac.com>:
>> Nice!
> \o/
>> 
>> Still needs to be done somewhere else though for performance.
>> Nothing at all wrong with this "patch", just SHOULD be done
>> deeper in dbiFindMatches() for highest performing, most general, etc etc.
> Yupp, there's still a lot room for improvement, I've tried to minimize
> the use of the patch as much as possible as is though, but of course it's
> not really very optimal several queries gets done using it..
> 
> Berkeley db is still a bit arcane to me, and whenever I start looking at
> wressling with it again after a couple of months not messing with,
> I spend way too much time having to refresh my memory again.. :|
> 

Well its not Berkeley DB but rather the pattern matching code for
partial key retrievals that is _VERY_ tricky.

> Oh well, this one at least works now and should finally put the last
> real issue with this migration for cooker to rest, one less thing to
> keep concerning myself about.. :p
> 

Yep. There's other issues such as generalizing the pattern matching
partial key retrievals that need to be attempted to. ATM the main
usage cases of Name/Nvra retrievals are wired up (and so the code
"works"). But you surely remember what happened when the same code
was permitted on Providename lookup's ... i.e. it the entire
access that needs to be looked at, not the partial key retrieval,
nor Berkeley DB, to ensure nothing breaks.

I.e. its the global context that needs to be examined carefully
and methodically ...

> Might just try optimize it a bit further later on.. :)
> 

Hint: get the test cases and the callgrind baseline in place first.

After that its just "wild hacking" until you get the desired functionality implemented.

73 de Jeff
["smime.p7s" (smime.p7s)]

0	*H
 010	+0	*H
 x00v n0
	*H
010	UIL10U

StartCom Ltd.1+0)U"Secure Digital Certificate Signing1806U/StartCom Class 1 \
Primary Intermediate Client CA0 100711153118Z
110712040640Z01 0U
224562-QYU8yAHNAs8olx1F10U
Persona Not Validated1)0'U StartCom Free Certificate Member10	*H
	
n3npq@mac.com0"0
	*H
0
ejh0fZ'FK`%sWY2 T)|#rbl$!;0lcZ-409ݛmi \
ht|ܯ"X~e!BńGbQ#e&o9̄K8ٛu)F>##x4Բ[{U4 \
WPf=jX}th˺RƀeEY$:)<0+*&9^/;MvV \
RQA00	U00U0U%0++0U
 e{12tPX^0U#0Sr풜\|~5NԸQ0U0
n3npq@mac.com0BU 90501+70 \
0.+"http://www.startssl.com/policy.pdf04+(http://www.startssl.com/intermediate.pdf0+00
 StartCom Ltd.0Limited Liability, see section *Legal Limitations* of the \
StartCom Certification Authority Policy available at \
http://www.startssl.com/policy.pdf0cU\0Z0+ ) \
'%http://www.startssl.com/crtu1-crl.crl0+ ) \
'%http://crl.startssl.com/crtu1-crl.crl0+009+0-http://ocsp.st \
artssl.com/sub/class1/client/ca0B+06http://www.startssl.com/certs/sub.class1.client.ca.crt0#U0http://www.startssl.com/0
 	*H
5˯4QN8V2T \
#1V0D8ZDHz=%m}1\5HE3V=	ɑ٪MdzC~"grHA>|Ǹ \
/2Dd<1](:ؔx&5gA(_Ka0cO%i1Lb?QT3~s@E}de} \
@<XMh,On,-ҫw3gumπ`:ϩ$ZYko00 
0
	*H
0}10	UIL10U

StartCom Ltd.1+0)U"Secure Digital Certificate Signing1)0'U StartCom \
Certification Authority0 071024210154Z
121022210154Z010	UIL10U

StartCom Ltd.1+0)U"Secure Digital Certificate Signing1806U/StartCom Class 1 \
Primary Intermediate Client CA0"0 	*H
0
	-).2AUGo#G
B|NDRpM-B=o-we5JQpa>O.# ._<V
[~**pz~3WG.ᘟMlr[<Ce6fqO"uxfWN#uic \
gkv$Lb%y`_{`xK'GN[0W0U00U0USr풜\|~5NԸQ0U# \
0N@[i04hCA򡁁0}10	UIL10U 
StartCom Ltd.1+0)U"Secure Digital Certificate Signing1)0'U StartCom \
Certification Authority0	U00=+10/0-+0!http://www.startssl.com/sfsca.crt0`UY0W0, \
* (&http://cert.startcom.org/sfsca-crl.crl0' % \
#!http://crl.startssl.com/sfsca.crl0]U \
T0P0L+70;0/+#http://cert.startcom.org/policy.pdf05+)http://cert.startcom.org/intermediate.pdf0+00' \
Start Commercial (StartCom) Ltd.0Limited Liability, read the section *Legal \
Limitations* of the StartCom Certification Authority Policy available at \
http://cert.startcom.org/policy.pdf0	`HB0P	`HB CAStartCom Class \
1 Primary Intermediate Free SSL Email Certificates0 	*H
[,0  &()04 \
~}ANӲ1	"ObT:d>ijkI:n$ir++vD 鄅GN$74 \
il*N5Y4eh.|ReF8!t|{t/`8CSa│R"܃.pi \
|sBI Z,&౼	=QCsvt⫆Qc:6"iAμy \
л'4sQ@@Vۉ0ǛHILƈ6FC8(P98'(V7 \
Ý3W4lV/KFQG 4u۪,2	l \
ZHx'%:8:(hP)/ܳjGBXbgnm%3:f \
9<_W1o0k0010	UIL10U 
StartCom Ltd.1+0)U"Secure Digital Certificate Signing1806U/StartCom Class 1 \
Primary Intermediate Client CAn0	+ 0	*H 	1	*H
0	*H
	1
110527151836Z0#	*H
	1^KW>)kxF0	+710010	UIL10U

StartCom Ltd.1+0)U"Secure Digital Certificate Signing1806U/StartCom Class 1 \
Primary Intermediate Client CAn0*H 	1 010	UIL10U

StartCom Ltd.1+0)U"Secure Digital Certificate Signing1806U/StartCom Class 1 \
Primary Intermediate Client CAn0 	*H
Omއ5idm87B51Wҩ @J \
w.8jZì!_ؿr,mސF!EcvouI='s> IcB[s \
W<c#aKHC54=s'G[f<2crF( \
#F7"R_Җ vE?n$+פG/jVnn\ \
?j,ݮʃԿeQ


______________________________________________________________________
RPM Package Manager                                    http://rpm5.org
Developer Communication List                        rpm-devel@rpm5.org

[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic