git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

author Alvaro Herrera <alvherre@alvh.no-ip.org>

2014年11月10日 18:56:08 +0000 (15:56 -0300)

committer Alvaro Herrera <alvherre@alvh.no-ip.org>

2014年11月10日 18:56:08 +0000 (15:56 -0300)

Besides a couple of typo fixes, per David Rowley, Thom Brown, and Amit
Langote, and mentions of BRIN in the general CREATE INDEX page again per
David, this includes silencing MSVC compiler warnings (thanks Microsoft)
and an additional variable initialization per Coverity scanner.

doc/src/sgml/brin.sgml patch | blob | blame | history

doc/src/sgml/ref/create_index.sgml patch | blob | blame | history

src/backend/access/brin/README patch | blob | blame | history

src/backend/access/brin/brin.c patch | blob | blame | history

src/backend/access/brin/brin_tuple.c patch | blob | blame | history

diff --git a/doc/src/sgml/brin.sgml b/doc/src/sgml/brin.sgml

index 03d1fd62783412e82bfbbd67a794f5ccebf36173..1ac282c57a62c3a263066d5c8a5b2753c5c8f597 100644 (file)

--- a/doc/src/sgml/brin.sgml

+++ b/doc/src/sgml/brin.sgml

@@ -64,7 +64,7 @@

<title>Built-in Operator Classes</title>

<para>

- The core <productname>PostgreSQL</productname> distribution includes

+ The core <productname>PostgreSQL</productname> distribution

includes the <acronym>BRIN</acronym> operator classes shown in

<xref linkend="brin-builtin-opclasses-table">.

</para>

diff --git a/doc/src/sgml/ref/create_index.sgml b/doc/src/sgml/ref/create_index.sgml

index ead33750fa246e3274d9ba7a4db640d2aa1c9eed..18bd0d33704d94f5edb44ae29cfd0a5e0bff5fa1 100644 (file)

--- a/doc/src/sgml/ref/create_index.sgml

+++ b/doc/src/sgml/ref/create_index.sgml

@@ -57,8 +57,8 @@ CREATE [ UNIQUE ] INDEX [ CONCURRENTLY ] [ [ IF NOT EXISTS ] <replaceable class=

<para>

<productname>PostgreSQL</productname> provides the index methods

- B-tree, hash, GiST, SP-GiST, and GIN. Users can also define their own index

- methods, but that is fairly complicated.

+ B-tree, hash, GiST, SP-GiST, GIN, and BRIN. Users can also define their own

+ index methods, but that is fairly complicated.

</para>

<para>

@@ -166,7 +166,8 @@ CREATE [ UNIQUE ] INDEX [ CONCURRENTLY ] [ [ IF NOT EXISTS ] <replaceable class=

<para>

The name of the index method to be used. Choices are

<literal>btree</literal>, <literal>hash</literal>,

- <literal>gist</literal>, <literal>spgist</> and <literal>gin</>.

+ <literal>gist</literal>, <literal>spgist</>, <literal>gin</>, and

+ <literal>brin</>.

The default method is <literal>btree</literal>.

</para>

</listitem>

@@ -492,7 +493,7 @@ Indexes:

</caution>

<para>

- Currently, only the B-tree, GiST and GIN index methods support

+ Currently, only the B-tree, GiST, GIN, and BRIN index methods support

multicolumn indexes. Up to 32 fields can be specified by default.

(This limit can be altered when building

<productname>PostgreSQL</productname>.) Only B-tree currently

diff --git a/src/backend/access/brin/README b/src/backend/access/brin/README

index 2619be8db560994bca22db217f498efebbedd1ad..636d96545b27a25d9c6329a0867c078d0d2d560c 100644 (file)

--- a/src/backend/access/brin/README

+++ b/src/backend/access/brin/README

@@ -126,7 +126,7 @@ that page range is complete and new tuples belong in a new page range that

hasn't yet been summarized. Those insertions do not create a new index

entry; instead, the page range remains unsummarized until later.

-Wehn VACUUM is run on the table, all unsummarized page ranges are

+Whenever VACUUM is run on the table, all unsummarized page ranges are

summarized. This action can also be invoked by the user via

brin_summarize_new_values(). Both these procedures scan all the

unsummarized ranges, and create a summary tuple. Again, this includes the

diff --git a/src/backend/access/brin/brin.c b/src/backend/access/brin/brin.c

index ae655497951a60342c4ad99762d8caa11073adb0..bd35cf6696acac2f207e6fb77d3076865d387d80 100644 (file)

--- a/src/backend/access/brin/brin.c

+++ b/src/backend/access/brin/brin.c

@@ -107,9 +107,11 @@ brininsert(PG_FUNCTION_ARGS)

BrinMemTuple *dtup;

BlockNumber heapBlk;

int keyno;

- BrinTuple *tmptup PG_USED_FOR_ASSERTS_ONLY;

- BrinMemTuple *tmpdtup PG_USED_FOR_ASSERTS_ONLY;

- Size tmpsiz PG_USED_FOR_ASSERTS_ONLY;

+#ifdef USE_ASSERT_CHECKING

+ BrinTuple *tmptup;

+ BrinMemTuple *tmpdtup;

+ Size tmpsiz;

+#endif

CHECK_FOR_INTERRUPTS();

diff --git a/src/backend/access/brin/brin_tuple.c b/src/backend/access/brin/brin_tuple.c

index d895cb715cba18a43a434aa9b81df596b487b414..af649c00afc277f3767f22d5aca5e5bd7dfbb2b1 100644 (file)

--- a/src/backend/access/brin/brin_tuple.c

+++ b/src/backend/access/brin/brin_tuple.c

@@ -95,7 +95,7 @@ brin_form_tuple(BrinDesc *brdesc, BlockNumber blkno, BrinMemTuple *tuple,

BrinTuple *rettuple;

int keyno;

int idxattno;

- uint16 phony_infomask;

+ uint16 phony_infomask = 0;

bits8 *phony_nullbitmap;

Size len,

hoff,

This is the main PostgreSQL git repository.

RSS Atom