On Mon, Jan 2, 2017 at 3:17 PM, Peter Geoghegan <pg@heroku.com> wrote:
> I noticed a bug in your latest revision:
>
>> + /*
>> + * In HJ_NEED_NEW_OUTER, we already selected the current inner batch for
>> + * reading from. If there is a shared hash table, we may have already
>> + * partially loaded the hash table in ExecHashJoinPreloadNextBatch.
>> + */
>> + Assert(hashtable->batch_reader.batchno = curbatch);
>> + Assert(hashtable->batch_reader.inner);
>
> Obviously this isn't supposed to be an assignment.
Right, thanks! I will post a new rebased version soon with that and
some other nearby problems fixed.
--
Thomas Munro
http://www.enterprisedb.com