not_bob
zzz: Thank you for fixing the OOM issue and adding a delay in loading torrents with i2psnark!
zzz
sure not_bob_afk nothing like an OOM to ruin your day
dr|z3d
zzz: git.idk.i2p/i2p-hackers/i2p.i2p/-/blob/master/router/java/src/net/i2p/router/peermanager/DBHistory.java#L44
dr|z3d
probably wants to be 60*60*100l
zzz
whataboutit?
dr|z3d
probably wants to be 60*60*100l
zzz
why?
dr|z3d
because you changed the stat frequency in ProfileOrganizer?
dr|z3d
java.lang.NullPointerException: Cannot invoke "net.i2p.stat.Rate.getCurrentTotalValue()" because "invalidReplyRate" is null
zzz
I didnt touch it
zzz
ProfileOrganizer.java: Rate invalidReplyRate = invalidReplyRateStat.getRate(30*60*1000l);
dr|z3d
ok, my bad, guess your rejigging of the values unturfed something I'd changed, like that rate.
zzz
all I did in that commit was move some code around
dr|z3d
yeah, not that commit, possibly the one where you junked a bunch of 30m stats.
dr|z3d_
yeah, nevermind then. maybe git.idk.i2p/i2p-hackers/i2p.i2p/-/commit/37beb532454b010eae904c5aca26848d8717c241 did some stuff that indirectly poked at some changes I'd made here.
dr|z3d_
(I removed most of the 30m stats a while back)
zzz
ok
dr|z3d_
I see it now.
dr|z3d_
you removed createRates(statGroup);
dr|z3d_
and added: _invalidReplyRate = new RateStat("dbHistory.invalidReplyRate", "How often does this peer give us a bad (nonexistant, forged, etc) peer?", statGroup, new long[] { 30*60*1000l });
dr|z3d_
whereas I'd set that at 60m previously. so no worries.
zzz
didnt add it, just moved it
dr|z3d
sure, and I missed the change in rate in the merge.
dr|z3d
you might disagree, but having a base value of 60s for all graphable stats gives you a common scale for the graphs.
zzz
it does, but at a memory/cpu cost
dr|z3d
always some tradeoff, but I haven't noticed much in the way of an unreasonable demand on either here, and most of the stats are @ 1m.