On Mon, Oct 12, 2015 at 12:47 AM, Peter Geoghegan <p...@heroku.com> wrote: > I also noticed that I failed to reset the last_returned strcoll() > cache variable as part of an abbreviation call, despite the fact that > tapesort may freely interleave conversions with comparisons, while > reusing buf1 and buf2 both as scratch space for strxfrm() blobs, as > well as for storing strings to be compared with strcoll(). I suggest > that the attached patch also be applied to fix this issue.
I think that I jumped the gun with this fix, because theoretically you can still get the same problem in the opposite direction -- an original string treated as a strxfrm() blob when the cache is consulted. I'll consider a more comprehensive fix. -- Peter Geoghegan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers