Re: [PATCH] Add tests for binaryheap.c

From: Aleksander Alekseev <aleksander(at)tigerdata(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Subject: Re: [PATCH] Add tests for binaryheap.c
Date: 2025-07-17 12:03:55
Message-ID: CAJ7c6TMxK4kN-5A13NThatFEBsTExsL4m9zE++UE69COZQLj2w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Nathan,

> > Make sense. Here is the corrected patch.
>
> I trimmed this down some more while trying to keep the coverage the same.

LGTM except for a slight issue with double semicolons at
test_binaryheap.c:160. Here is the updated patch.

Attachment Content-Type Size
v4-0001-Add-tests-for-binaryheap.c.patch text/x-patch 13.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2025-07-17 12:11:08 Re: generic plans and "initial" pruning
Previous Message Álvaro Herrera 2025-07-17 12:01:06 Re: [Feature Request] Support INSERT ... FROM ... TO for date ranges