On 12/4/17 22:15, Noah Misch wrote: > While reviewing this patch, I noticed a couple of nearby defects: > > - RelationFindReplTupleSeq() says "Note that this stops on the first matching > tuple.", but that's not the case. It visits every row in the table, and it > uses the last match. The claimed behavior sounds more attractive.
Yeah, that looks like a mistake. > - RelationFindReplTupleSeq() has comment "/* Start an index scan. */", an > inapplicable copy-paste from RelationFindReplTupleByIndex(). I have fixed that. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
