Make the join access types into RowIterators, unifying the two abstractions.
This allows us to finally split join_init_read_record() into a function that sets
up classes etc. without actually doing work, which further separates optimizer
and executor.

Change-Id: I24dbfc5d9ad0be9236d91182985a8284e6a1b6a5