Justin Berman

I think he's re-importing into the same wallet that exports (not doing the cold signing flow). One thing I've noticed so far doing this that's off (that I'm still looking...

> Maybe the best is to error out when importing outputs in a hot wallet. And presumably importing key images in a cold wallet. I agree this seems like the...

@woodser can you see if you can repro the hanging with #8510 :)

Done reviewing @rbrunner7 :) The 2 main things in my review that I don't think had anything to do with the location of code were: 1) [re-visiting the idea](https://github.com/monero-project/monero/pull/8076#pullrequestreview-887976909) that...

~~The daemon could also keep track of time of last update to the container, pass this to client, and client passes this back to the daemon along with the counter....

I'd imagine implementing in LMDB + using a circular queue where elements are stored in the db instead of in memory is a reasonable solution, albeit fairly complex (no need...

Actually the issue in October I believe was large transactions with many inputs, not a large volume of transactions. So I don't think this PR as is would have really...

Is the daemon using the tor tx proxy/do you have logs from the daemon? It sounds like the daemon is failing to broadcast the tx to the rest of the...

Looks like the same thing I was running into to me! And duh.. seems `relay_tx` is the daemon RPC endpoint that should enable re-submitting the tx. I think there could...

It seems both [`relay_tx`](https://github.com/monero-project/monero/blob/f49fc9b4876382d5c6f08fd7a6125b554c49e260/src/rpc/core_rpc_server.h#L179) and [`flush_txpool`](https://github.com/monero-project/monero/blob/f49fc9b4876382d5c6f08fd7a6125b554c49e260/src/rpc/core_rpc_server.h#L173) are restricted endpoints. `flush_txpool` makes sense to me to keep restricted for DoS protection (edit: can't allow anyone to cancel transactions in your pool)....