Comment 2 for bug 1855329

While I see similar numbers on Pg 9.6 that I saw on Pg 9.5 with both versions of this fix, the numbers are so close that the difference is negligible. What I see is that some parts of the NOT NULL version are faster than the NULLS LAST version, but the planning step seems to be longer on the NOT NULL version. Again, the difference that I see is about 1ms on average.

I've added my signoff to the branch below. I'm not certain if we want to add release notes for this change or not. Another pair eyes would be useful, particularly from a site that sees a larger difference in the numbers.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dyrcona/lp-1855329-hold_shelf_speed-signoff