| From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
|---|---|
| To: | Shawn Debnath <sdn(at)amazon(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Adding SMGR discriminator to buffer tags |
| Date: | 2019-07-11 22:16:21 |
| Message-ID: | CA+hUKGJ+qr=9JwBm0+Lgv7Vmv1Zm1mpyODJ9oB7wy6bN25pLtw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, May 10, 2019 at 8:54 AM Shawn Debnath <sdn(at)amazon(dot)com> wrote:
> On Wed, May 08, 2019 at 06:31:04PM +1200, Thomas Munro wrote:
> Looks good to me. Minor nit: update the comment for XLogRecGetBlockTag:
Fixed. Also fixed broken upgrade scripts for pg_buffercache
extension, as pointed out by Robert[1] on the main thread where undo
stuff is being discussed. Attempts to keep subtopics separated have so
far failed, so the thread ostensibly about orphaned file cleanup is
now about undo work allocation, but I figured it'd be useful to
highlight this patch separately as it'll be the first to go in, and
it's needed by your work Shawn. So I hope we're still on the same
page with this refactoring patch.
One thing I'm not sure about is the TODO message in parsexlog.c's
extractPageInfo() function.
--
Thomas Munro
https://enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| 0002-Move-tablespace-dir-creation-from-smgr.c-to-md.c.patch | application/octet-stream | 2.8 KB |
| 0001-Add-SmgrId-to-smgropen-and-BufferTag.patch | application/octet-stream | 65.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2019-07-11 22:37:41 | Re: [Proposal] Table-level Transparent Data Encryption (TDE) and Key Management Service (KMS) |
| Previous Message | Tom Lane | 2019-07-11 21:28:27 | Re: base backup client as auxiliary backend process |