luigi1111w: I feel like we will have to wait until you are back so that we can have more time to review and test

@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

What do you think of it?

It doesn't make changes to for ZMQ optimization but IMHO that should be a different PR

> it also fixes connection dropping for fees that are too low

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

selsta kk

jeffro256[m] do ZMQ fix a second commit too, wtf

why are you making everyone do more work

I'm working on a totally different fix now, do waste my time

*don't waste my time

sech1: b/c it's a better fix

just do a second commit with ZMQ

I'm not going to make a separate PR just because "it's more right"

I'm working on a different fix already while you're obsessed with "by the book"

Did you miss the part where my PR in question fixed more than tx_extra?

It's just 4 lines of code

Just look what bug I found: p2pool.io/u/d70f28419ee2a43d/image.png

I don't care about that PR anymore

8808 is ok as long as you add ZMQ fix

Soon I'll find out how many threads it's allowed to create per process in Windows, lol :D

hyc monero-project/monero #8810

.merges

8811, 8813 (and their copies for master branch) need a review

I mean one more reviewer

.merge+ 8810 8811 8808 8813

Added