On Tue, 2 Jul 2024 at 10:15, Aleksander Alekseev
wrote:
> The referred patch was rejected at first because it didn't modify
> nodeSeqScan.c to make use of the change within the core.
I guess we interpret Heikis email differently. I read it as: "If this
improves performance, then let's also start
Hi,
> > As I recall, previously it was argued that changes like this should
> > have some use within the core [1].
>
> I don't see that argument anywhere in the thread honestly. I did see
> heiki asking why it would be useful for extensions, but that was
> answered there.
The referred patch was r
On Mon, 1 Jul 2024 at 15:48, Aleksander Alekseev
wrote:
> As I recall, previously it was argued that changes like this should
> have some use within the core [1].
I don't see that argument anywhere in the thread honestly. I did see
heiki asking why it would be useful for extensions, but that was
Hi,
> This patches changes the HeapKeyTest macro to add handling for SK_SEARCHNULL
> and SK_SEARCHNOTNULL. While currently no core codes uses these ScanKey flags
> it would be useful for extensions if it was supported so extensions
> dont have to implement
> handling for those by themself.
As I r
, Sven Klemm
From 7ba786ebc065a27375dac9ced6d1928d0e8e5740 Mon Sep 17 00:00:00 2001
From: Sven Klemm
Date: Sat, 22 Jun 2024 15:21:53 +0200
Subject: [PATCH] Handle SK_SEARCHNULL and SK_SEARCHNOTNULL in HeapKeyTest
Change HeapKeyTest to not return false on SK_ISNULL but instead
return a result