On Mon, Jan 2, 2017 at 3:17 PM, Peter Geoghegan <p...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers