< MarcoFalke>
#proposedmeetingtopic add #19629 to high prio, discuss whether to remove the pulls that need rebase from high prio. (MarcoFalke, but won't be around)
< wumpus>
MarcoFalke: sure, let's just add it, I prefer doing high prio list changes only during the meeting but also understand not everyone can be there at that time
< achow101>
#proposedmeetingtopic: what to do about zapwallettxes
< wumpus>
achow101: I think zapwallettx was introduced for the same reason as abandontransaction
< achow101>
wumpus: it seems like that's been the only use for it
< wumpus>
I'm not sure people are actually ever using it
< wumpus>
right
< achow101>
and both don't work with -persistmempool
< wumpus>
I think abandontransaction, to be able to abandon specific conflicted or stuck transaction, is superior in every case
< wumpus>
but I'm not sure, it might be zapwallettx is more of a sledge hammer that can be useful in some cases
< sipa>
i think zapwallettxes was also intended as a way to.recover from very corrupted wallet files
< sipa>
or is that savagewallet
< wumpus>
that ^^
< achow101>
there was a test case MarcoFalke made recently that did savagewallet and zapwallettxes would fix the problem
< wumpus>
salvagewallet already has multiple settings IIRC
< wumpus>
I guess zapwallettxes could be another salvagewallet level
< achow101>
it was something like savagewallet ended up adding a tx to the wallet multiple times, so zapwallettxs could fix that with dumping all the txs
< wumpus>
salvagewallet is ugly and unreliable as it goes over the raw database file data, so it sometimes misinterprets things