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
