| From: | "Bossart, Nathan" <bossartn(at)amazon(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, Christoph Berg <myon(at)debian(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_ls_tmpdir() |
| Date: | 2018-10-02 19:01:30 |
| Message-ID: | 7927BD34-5A62-4F51-A39E-B89FD81D4A43@amazon.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 10/2/18, 11:46 AM, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> "Bossart, Nathan" <bossartn(at)amazon(dot)com> writes:
>> On 10/2/18, 7:22 AM, "Andrew Dunstan" <andrew(dot)dunstan(at)2ndquadrant(dot)com> wrote:
>>> See the bottom of src/backend/catalog/system_views.sql starting around
>>> line 1010.
>
>> AFAICT the cleanest way to do this in system_views.sql is to hard-code
>> the pg_default tablespace OID in the DEFAULT expression. So, it might
>> be best to use the two function approach if we want pg_ls_tmpdir() to
>> default to the pg_default tablespace.
>
> That would be pretty bletcherous, so +1 for just making two C functions.
Alright, here's an updated patch.
Nathan
| Attachment | Content-Type | Size |
|---|---|---|
| pg_ls_tmpdir_v3.patch | application/octet-stream | 7.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Marco Atzeri | 2018-10-02 19:20:06 | Re: Cygwin linking rules |
| Previous Message | Tom Lane | 2018-10-02 18:55:29 | Re: SerializeParamList vs machines with strict alignment |