@eyedeekay
&zzz
+FreefallHeavens_
+R4SAS
+RN_
+Romster_
+StormyCloud
+acetone
+cims
+eche|off
+hagen
+nilbog
+nyaa2pguy
+orignal
+postman
+qend-irc2p
+snex
Arch
Danny
Holmes
Irc2PGuest14026
Irc2PGuest26254
Irc2PGuest28384
Irc2PGuest36664
OfficialCIA
Onn4l7h
Onn4|7h
Over
Sleepy
T3s|4_
U1F642
Zapek2
aargh4
ac9f
ahiru
ananas
anontor
dr4wd3
eyedeekay_
eyedeekay_bnc
leopold_
mahlay
makoto
marek
mareki2p_
n2_
newbie|2
not_bob_afk2
pinotto
poriori
profetikla
r00tobo
rapidash
test7363673
uop23ip
urist_
x74a6
zelgomer
zzz
dr|z3d, obscuratus, proposed patch is up on gitlab
obscuratus
zzz: Thanks, I'll throw it up on my testing network.
dr|z3d
thanks, zzz, will take it for a spin.
dr|z3d
does this negate obscuratus' first LookupDestJob path, or can they be run side-by-side?
dr|z3d
*patch
zzz
the former
dr|z3d
ok, thanks
dr|z3d
patch looks good, zzz. easy way to test is to hit some of the b32 links on notbob, they should resolve immediately.
dr|z3d
(previously hitting a b32 would fail on first attempt)
obscuratus
zzz: The patch is working well so far on my testing network. Examining my logs, I can see the LS hitting the persistent data store much earlier, and the subsequent lookups are all successful.
obscuratus
I haven't found any errors or observable regressions so far.
zzz
thanks guys, I'll probably let it soak another day or two
dr|z3d
speaking of soaking, zzz, git.idk.i2p/i2p-hackers/i2p.i2p/-/merge_requests/49 ? :)
zzz
the zlatinb-proposed test regimen kinda squashed it
dr|z3d
oh :(