public inbox for [email protected]
help / color / mirror / Atom feedFrom: Alexander Law <[email protected]>
To: Peter Eisentraut <[email protected]>
To: [email protected] <[email protected]>
Subject: Re: Some minor error fixes
Date: Wed, 4 May 2016 22:04:59 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-docs>
Thank you!
I have some more errors written down, maybe they are worth fixing too.
Second patch is for consistency in [1].
(I think XMLValidate could be aligned with "IS VALID predicate")
Third patch is for language name teared down in [1].
It seems that root of this typo is as far as in SQL Standard (see M015
description in [2]), but IMO it should be fixed anyway.
[1]
http://www.postgresql.org/docs/9.5/static/unsupported-features-sql-standard.html
[2]
http://jtc1sc32.org/doc/N1501-1550/32N1529T-text%20for%20FCD%209075-9.pdf
04.05.2016 21:11, Peter Eisentraut пишет:
> On 5/4/16 10:40 AM, Alexander Law wrote:
>> Please have a look at the some more errors.
>> Regarding second patch, I think, that inconsistency should be fixed by
>> omitting '_name', as this parameter could also include a password.
>
> fixed
>
--
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
Attachments:
[text/x-patch] 1-fix-gin-typo.patch (750B, 2-1-fix-gin-typo.patch)
download | inline diff:
diff --git a/doc/src/sgml/gin.sgml b/doc/src/sgml/gin.sgml
index a392f94..b805af5 100644
--- a/doc/src/sgml/gin.sgml
+++ b/doc/src/sgml/gin.sgml
@@ -539,7 +539,7 @@
<function>extractQuery</> to pass additional data to the
<function>consistent</> and <function>comparePartial</> methods.
To use it, <function>extractQuery</> must allocate
- an array of <literal>*nkeys</> Pointers and store its address at
+ an array of <literal>*nkeys</> pointers and store its address at
<literal>*extra_data</>, then store whatever it wants to into the
individual pointers. The variable is initialized to <symbol>NULL</symbol> before
call, so this argument can simply be ignored by operator classes that
[text/x-patch] 2-sql-features-improve-consistency.patch (816B, 3-2-sql-features-improve-consistency.patch)
download | inline diff:
diff --git a/src/backend/catalog/sql_features.txt b/src/backend/catalog/sql_features.txt
index fd9e78d..71c08af 100644
--- a/src/backend/catalog/sql_features.txt
+++ b/src/backend/catalog/sql_features.txt
@@ -656,10 +656,10 @@ X271 XMLValidate: data-driven case NO
X272 XMLValidate: ACCORDING TO clause NO
X273 XMLValidate: ELEMENT clause NO
X274 XMLValidate: schema location NO
-X281 XMLValidate: with DOCUMENT option NO
+X281 XMLValidate with DOCUMENT option NO
X282 XMLValidate with CONTENT option NO
X283 XMLValidate with SEQUENCE option NO
-X284 XMLValidate NAMESPACE without ELEMENT clause NO
+X284 XMLValidate: NAMESPACE without ELEMENT clause NO
X286 XMLValidate: NO NAMESPACE with ELEMENT clause NO
X300 XMLTable NO
X301 XMLTable: derived column list option NO
[text/x-patch] 3-fix-m015-feature-description.patch (530B, 4-3-fix-m015-feature-description.patch)
download | inline diff:
diff --git a/src/backend/catalog/sql_features.txt b/src/backend/catalog/sql_features.txt
index 71c08af..569a43f 100644
--- a/src/backend/catalog/sql_features.txt
+++ b/src/backend/catalog/sql_features.txt
@@ -520,7 +520,7 @@ M011 Datalinks via Ada NO
M012 Datalinks via C NO
M013 Datalinks via COBOL NO
M014 Datalinks via Fortran NO
-M015 Datalinks via M NO
+M015 Datalinks via MUMPS NO
M016 Datalinks via Pascal NO
M017 Datalinks via PL/I NO
M018 Foreign data wrapper interface routines in Ada NO
view thread (11+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: Some minor error fixes
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox