Use FlushRecPtr, not WriteRecPtr, to determine start/stop points

…otherwise we could end up depending on WAL records that haven't yet
been written to disk. Not a big deal in testing, but this commit is
the right way to do it.
Throw an error if slot_name isn't defined in stop_logical_replication()