we require wal_level=logical, would be nice to support looser in future

agreed that there's a huge complexity cost to going this route