diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2015-03-10 12:26:34 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2015-03-10 12:27:15 -0300 |
| commit | e491bd2ee34860b14ff18abc5602f9aa5b197a2d (patch) | |
| tree | b6b2bde52149169db4b0c666bce203c7d8da1820 /src/include/access/spgist_private.h | |
| parent | 865f14a2d31af23a05bbf2df04c274629c5d5c4d (diff) | |
| download | postgresql-e491bd2ee34860b14ff18abc5602f9aa5b197a2d.tar.gz postgresql-e491bd2ee34860b14ff18abc5602f9aa5b197a2d.zip | |
Move BRIN page type to page's last two bytes
... which is the usual convention among AMs, so that pg_filedump and
similar utilities can tell apart pages of different AMs. It was also
the intent of the original code, but I failed to realize that alignment
considerations would move the whole thing to the previous-to-last word
in the page.
The new definition of the associated macro makes surrounding code a bit
leaner, too.
Per note from Heikki at
http://www.postgresql.org/message-id/546A16EF.9070005@vmware.com
Diffstat (limited to 'src/include/access/spgist_private.h')
| -rw-r--r-- | src/include/access/spgist_private.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/include/access/spgist_private.h b/src/include/access/spgist_private.h index 0492ef6114b..413f71e7298 100644 --- a/src/include/access/spgist_private.h +++ b/src/include/access/spgist_private.h @@ -64,6 +64,8 @@ typedef SpGistPageOpaqueData *SpGistPageOpaque; * which otherwise would have a hard time telling pages of different index * types apart. It should be the last 2 bytes on the page. This is more or * less "free" due to alignment considerations. + * + * See comments above GinPageOpaqueData. */ #define SPGIST_PAGE_ID 0xFF82 |