I didn't look at this patch, but, yesterday, I checked in a change that must be essentially the same. I fixed up the import issues, and, as I recall, reversed the order of the process() and process_request() defs. I think it is a good change because the drivers can drop those imports, and it also clears the path for those that want customized requests.