Hi,
I've edited the patch a bit more and pushed it:
- I ended up not liking _impl() that much, and changed it to common. That
seems more in line what it actually does.
- All the beginscan functions are called beginscan, so I renamed
table_scan_begin_impl to match
- We normally don't use function names in error message strings. I've
rephrased the error
- Some comment rephrasing
Greetings,
Andres