On Tue, Aug 27, 2019 at 8:30 PM Michael Paquier <mich...@paquier.xyz> wrote: > Please note that I have not looked in details where we could put that, > but perhaps Robert and Peter G who worked on 4ea51cd to add support > for abbreviated keys have some ideas, so I am adding them in CC for > input.
Movead is correct -- the NULL handling within ApplySortAbbrevFullComparator() cannot actually be used currently. I wouldn't change anything about the code, though, since it's useful to defensively handle NULLs. -- Peter Geoghegan
