00:12:21 <selsta> luigi1111w: I feel like we will have to wait until you are back so that we can have more time to review and test
00:12:26 <jeffro256[m]> @sech1 Okay I trimmed down my alternative PR to its absolute smallest (not including comments), and I made you co-author . I don't want to start a pissing contest over trivial things, but I prefer my PR because it also fixes connection dropping for  fees that are too low 
00:12:42 <jeffro256[m]> What do you think of it?
00:13:16 <jeffro256[m]> It doesn't make changes to for ZMQ optimization but IMHO that should be a different PR 
00:15:20 <jeffro256[m]> > it also fixes connection dropping for  fees that are too low
00:15:54 <jeffro256[m]> And could be easily extended to other relay only rules by adding one line of code in the tx pool, without having to touch the protocol handler each time a change is made 
01:02:05 <luigi1111w> selsta kk
05:17:44 <sech1> jeffro256[m] do ZMQ fix a second commit too, wtf
05:17:53 <sech1> why are you making everyone do more work
05:18:03 <sech1> I'm working on a totally different fix now, do waste my time
05:18:14 <sech1> *don't waste my time
05:18:31 <jeffro256[m]> sech1: b/c it's a better  fix 
05:18:56 <sech1> just do a second commit with ZMQ
05:19:09 <sech1> I'm not going to make a separate PR just because "it's more right"
05:19:25 <sech1> I'm working on a different fix already while you're obsessed with "by the book"
05:20:31 <jeffro256[m]> Did you miss the part where my PR in question fixed more than tx_extra?
05:20:41 <jeffro256[m]> It's just 4 lines of code 
05:20:41 <sech1> Just look what bug I found:  https://p2pool.io/u/d70f28419ee2a43d/image.png 
05:20:49 <sech1> I don't care about that PR anymore
05:21:15 <sech1> 8808 is ok as long as you add ZMQ fix
05:22:13 <sech1> Soon I'll find out how many threads it's allowed to create per process in Windows, lol :D
06:50:00 <sech1> hyc https://github.com/monero-project/monero/pull/8810
08:22:52 <sech1> .merges
08:22:52 -xmr-pr- 8794 8805
08:23:57 <sech1> 8811, 8813 (and their copies for master branch) need a review
08:24:14 <sech1> I mean one more reviewer
12:22:37 <selsta> .merge+ 8810 8811 8808 8813
12:22:37 <xmr-pr> Added