Fwiw, I have a thing about TODOs checked into code.  TODO statements are indications of things left to do before the person who is currently implementing something can commit their code.  Once they are checked into the repository, everyone else often doesn't have enough context to fully understand them.  It's a bit of a nit, but can you change this to just

// If we wanted to recover from a lost connection, this is where we would do it.  We have chosen to punt on that for now though.