So `Ecto.Adapter.start_link` actually receives the already parsed url so this one feels a bit inconsistent because it needs to parse itself internally. I would say to leave the parsing out of the adapter but I don't have a strong argument for such.