well then isnt he already good if it isnt even showing up on block explorers?
Not really, he's using bitcoin core... Core will rebroadcast his transaction if it's restarted. When he abandons his transaction, or zaps his wallet, core will no longer rebroadcast
Also, after zapping or abandoning, it'll be much easyer to re-use the unspent outputs when creating a new transaction with a higher fee.
Your approach was OK tough, you're right: if the transaction is improperly propagated, the OP has a slim chance of getting his tx in a block anyways, but that was actually the thing i was aiming at in the first place.
If you leave it like this, you'll probably keep rebroadcasting your transaction forever... using abandontransaction or zapwallettxes isn't hard at all, i've given you the exact command you should enter in your windows command prompt...
If it's just a transaction between your own wallets, this might be "fine" to leave it like this tough... Sooner or later, you'll probably end up spending the unspent output again in a new transaction, or maybe you'll accidently do a CPFP, or maybe the fee drops to a point where your transaction WILL end up in a block, or maybe a rescan is triggered after some problem, and your tx gets zapped... As long as both wallets belong to you AND you don't need your funds right away, you might get away with it.