Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name

From: Yurii Rashkovskii <yrashk(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: Aleksander Alekseev <aleksander(at)timescale(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name
Date: 2023-07-04 01:00:12
Message-ID: CA+RLCQy+rP+KdhjvNkUXwToVwO=pWrhDFqqaMKpmRpGKh+Ns8w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Nathan,

On Mon, Jul 3, 2023 at 3:08 PM Nathan Bossart <nathandbossart(at)gmail(dot)com>
wrote:

> On Sun, Jul 02, 2023 at 04:37:52PM -0700, Yurii Rashkovskii wrote:
> > Thank you for revising the patch. While this is relatively minor, I think
> > it should be set to MAXPGPATH directly to clarify their relationship.
>
> Committed. I set the size to MAXPGPATH directly instead of inventing a new
> macro with the same value.
>

Great, thank you! The reason I was leaving the other constant in place to
make upgrading extensions trivial (so that they don't need to adjust for
this), but if you think this is a better way, I am fine with it.

--
Y.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message James Coleman 2023-07-04 01:12:58 pgindent (probably my missing something obvious)
Previous Message Richard Guo 2023-07-04 00:48:00 Re: Improve list manipulation in several places