pg.ddx.io pgsql-hackers@postgresql.org mailing list archive
help / color / mirror / Atom feeddocs: note ownership requirement for refreshing materialized views
15+ messages / 5 participants
[nested] [flat]
Commits from this thread:
2018-08-17 ee80124811 b43cf1dcde 0dfaf8f763 86e873c016 82784f0889 777192f0dd b6c994af1f Mention ownership requirements for REFRESH MATERIALIZED VIEW in docs
* docs: note ownership requirement for refreshing materialized views
@ 2018-08-15 23:46 Dian Fay <dian.m.fay@gmail.com>
0 siblings, 2 replies; 15+ messages in thread
From: Dian Fay @ 2018-08-15 23:46 UTC (permalink / raw)
To: pgsql-hackers@lists.postgresql.org
hi all! I discovered today that the REFRESH MATERIALIZED VIEW
documentation doesn't mention that only the owner (or a superuser) may
actually perform the refresh operation. This patch adds a note to that
effect.
Dian Fay
Attachments:
0001-docs-note-ownership-requirement-for-refreshing-mater.patch (text/x-patch, 572B) download
diff --git a/doc/src/sgml/ref/refresh_materialized_view.sgml b/doc/src/sgml/ref/refresh_materialized_view.sgml
index 9cf01a25a5..f001a661e1 100644
--- a/doc/src/sgml/ref/refresh_materialized_view.sgml
+++ b/doc/src/sgml/ref/refresh_materialized_view.sgml
@@ -91,6 +91,10 @@ REFRESH MATERIALIZED VIEW [ CONCURRENTLY ] <replaceable class="parameter">name</
<refsect1>
<title>Notes</title>
+ <para>
+ Only the owner of a materialized view or a superuser can refresh it.
+ </para>
+
<para>
While the default index for future
<xref linkend="sql-cluster"/>
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 01:03 Michael Paquier <michael@paquier.xyz>
parent: Dian Fay <dian.m.fay@gmail.com>
1 sibling, 1 reply; 15+ messages in thread
From: Michael Paquier @ 2018-08-16 01:03 UTC (permalink / raw)
To: Dian Fay <dian.m.fay@gmail.com>; +Cc: pgsql-hackers@lists.postgresql.org
On Wed, Aug 15, 2018 at 07:46:49PM -0400, Dian Fay wrote:
> hi all! I discovered today that the REFRESH MATERIALIZED VIEW documentation
> doesn't mention that only the owner (or a superuser) may actually perform
> the refresh operation. This patch adds a note to that effect.
I think that's a good idea. I would rewrite that a bit differently, like:
To refresh a materialized view, one must be the materalized view's owner
or a superuser.
Thougts or objections?
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 01:06 Jonathan S. Katz <jonathan.katz@excoventures.com>
parent: Dian Fay <dian.m.fay@gmail.com>
1 sibling, 1 reply; 15+ messages in thread
From: Jonathan S. Katz @ 2018-08-16 01:06 UTC (permalink / raw)
To: Dian Fay <dian.m.fay@gmail.com>; +Cc: pgsql-hackers@lists.postgresql.org
Hi Dian,
> On Aug 15, 2018, at 7:46 PM, Dian Fay <dian.m.fay@gmail.com> wrote:
>
> hi all! I discovered today that the REFRESH MATERIALIZED VIEW documentation doesn't mention that only the owner (or a superuser) may actually perform the refresh operation. This patch adds a note to that effect.
I played around with this feature a bit and did see this was the case.
Also while playing around I noticed the error message was as such:
test=> REFRESH MATERIALIZED VIEW blah;
ERROR: must be owner of relation blah
But it’s not a relation, it’s a materialized view. I attached a patch
that I think should fix this. Kudos to Dave Cramer who was
sitting next to me helping me to locate files and confirm assumptions.
Jonathan
Attachments:
0001-treat-refresh-mat-view-as-mat-view.patch (application/octet-stream, 956B) download
diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 87f5e95827..a769e38829 100644
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -4138,6 +4138,7 @@ RefreshMatViewStmt:
REFRESH MATERIALIZED VIEW opt_concurrently qualified_name opt_with_data
{
RefreshMatViewStmt *n = makeNode(RefreshMatViewStmt);
+ n->relkind = OBJECT_MATVIEW;
n->concurrent = $4;
n->relation = $5;
n->skipData = !($6);
diff --git a/src/include/nodes/parsenodes.h b/src/include/nodes/parsenodes.h
index 07ab1a3dde..2998f401bb 100644
--- a/src/include/nodes/parsenodes.h
+++ b/src/include/nodes/parsenodes.h
@@ -3217,6 +3217,7 @@ typedef struct RefreshMatViewStmt
bool concurrent; /* allow concurrent access? */
bool skipData; /* true for WITH NO DATA */
RangeVar *relation; /* relation to insert into */
+ ObjectType relkind; /* OBJECT_MATVIEW */
} RefreshMatViewStmt;
/* ----------------------
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 01:15 Michael Paquier <michael@paquier.xyz>
parent: Jonathan S. Katz <jonathan.katz@excoventures.com>
0 siblings, 1 reply; 15+ messages in thread
From: Michael Paquier @ 2018-08-16 01:15 UTC (permalink / raw)
To: Jonathan S. Katz <jonathan.katz@excoventures.com>; +Cc: Dian Fay <dian.m.fay@gmail.com>; pgsql-hackers@lists.postgresql.org
On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
> I played around with this feature a bit and did see this was the case.
> Also while playing around I noticed the error message was as such:
>
> test=> REFRESH MATERIALIZED VIEW blah;
> ERROR: must be owner of relation blah
>
> But it’s not a relation, it’s a materialized view. I attached a patch
> that I think should fix this. Kudos to Dave Cramer who was
> sitting next to me helping me to locate files and confirm assumptions.
A relation may be a materialized view, no? The ACL check happens in
RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
matview.c).
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 01:44 Dian Fay <dian.m.fay@gmail.com>
parent: Michael Paquier <michael@paquier.xyz>
0 siblings, 1 reply; 15+ messages in thread
From: Dian Fay @ 2018-08-16 01:44 UTC (permalink / raw)
To: Michael Paquier <michael@paquier.xyz>; +Cc: pgsql-hackers@lists.postgresql.org
I feel resorting to the infinitive asks more involvement of the reader,
while leading with the responsible role(s) helps shortcut the process of
determining whether what follows is relevant. Efficiency is always a
virtue, although this is admittedly more than a little academic for a
one-sentence addition!
On 8/15/18 9:03 PM, Michael Paquier wrote:
> On Wed, Aug 15, 2018 at 07:46:49PM -0400, Dian Fay wrote:
>> hi all! I discovered today that the REFRESH MATERIALIZED VIEW documentation
>> doesn't mention that only the owner (or a superuser) may actually perform
>> the refresh operation. This patch adds a note to that effect.
> I think that's a good idea. I would rewrite that a bit differently, like:
> To refresh a materialized view, one must be the materalized view's owner
> or a superuser.
>
> Thougts or objections?
> --
> Michael
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 04:07 Tom Lane <tgl@sss.pgh.pa.us>
parent: Dian Fay <dian.m.fay@gmail.com>
0 siblings, 1 reply; 15+ messages in thread
From: Tom Lane @ 2018-08-16 04:07 UTC (permalink / raw)
To: Dian Fay <dian.m.fay@gmail.com>; +Cc: Michael Paquier <michael@paquier.xyz>; pgsql-hackers@lists.postgresql.org
Dian Fay <dian.m.fay@gmail.com> writes:
> I feel resorting to the infinitive asks more involvement of the reader,
> while leading with the responsible role(s) helps shortcut the process of
> determining whether what follows is relevant. Efficiency is always a
> virtue, although this is admittedly more than a little academic for a
> one-sentence addition!
I think Michael's point is that this formulation is unlike what we have
elsewhere for similar statements. Looking around, it seems like the
typical phraseology would be more like
"You must own the materialized view to use REFRESH MATERIALIZED VIEW."
It is not really customary to call out the superuser exception
explicitly, because if we did, we'd be mentioning superusers in every
other sentence. The point is covered by existing documentation that
says something to the effect of superusers bypassing all permissions
checks; and I think there's also a statement somewhere about superusers
implicitly being members of every role, which is a different way of
arriving at the same conclusion.
In any case, it's definitely an oversight that the REFRESH reference
page fails to address permissions at all. +1 for adding something.
regards, tom lane
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 05:05 Jonathan S. Katz <jonathan.katz@excoventures.com>
parent: Michael Paquier <michael@paquier.xyz>
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan S. Katz @ 2018-08-16 05:05 UTC (permalink / raw)
To: Michael Paquier <michael@paquier.xyz>; +Cc: Dian Fay <dian.m.fay@gmail.com>; pgsql-hackers@lists.postgresql.org
> On Aug 15, 2018, at 9:15 PM, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
>> I played around with this feature a bit and did see this was the case.
>> Also while playing around I noticed the error message was as such:
>>
>> test=> REFRESH MATERIALIZED VIEW blah;
>> ERROR: must be owner of relation blah
>>
>> But it’s not a relation, it’s a materialized view. I attached a patch
>> that I think should fix this. Kudos to Dave Cramer who was
>> sitting next to me helping me to locate files and confirm assumptions.
>
> A relation may be a materialized view, no? The ACL check happens in
> RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
> matview.c).
Comment on the RangeVarCallbackOwnsTable func (abbr):
/*
* This is intended as a callback for RangeVarGetRelidExtended(). It allows
* the relation to be locked only if (1) it's a plain table, materialized
* view, or TOAST table and (2) the current user is the owner (or the
* superuser). This meets the permission-checking needs of CLUSTER, REINDEX
* TABLE, and REFRESH MATERIALIZED VIEW; we expose it here so that it can be
* used by all.
*/
So it’s sharing the permission checking needs amongst all of those commands.
As a user I could be confused if I saw the above error message, esp. because
the behavior of REFRESH .. is specific to materialized views.
Jonathan
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 13:36 Dian Fay <dian.m.fay@gmail.com>
parent: Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 15+ messages in thread
From: Dian Fay @ 2018-08-16 13:36 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Michael Paquier <michael@paquier.xyz>; pgsql-hackers@lists.postgresql.org
Fair enough! Here's a new version.
On 8/16/18 12:07 AM, Tom Lane wrote:
> Dian Fay <dian.m.fay@gmail.com> writes:
>> I feel resorting to the infinitive asks more involvement of the reader,
>> while leading with the responsible role(s) helps shortcut the process of
>> determining whether what follows is relevant. Efficiency is always a
>> virtue, although this is admittedly more than a little academic for a
>> one-sentence addition!
> I think Michael's point is that this formulation is unlike what we have
> elsewhere for similar statements. Looking around, it seems like the
> typical phraseology would be more like
>
> "You must own the materialized view to use REFRESH MATERIALIZED VIEW."
>
> It is not really customary to call out the superuser exception
> explicitly, because if we did, we'd be mentioning superusers in every
> other sentence. The point is covered by existing documentation that
> says something to the effect of superusers bypassing all permissions
> checks; and I think there's also a statement somewhere about superusers
> implicitly being members of every role, which is a different way of
> arriving at the same conclusion.
>
> In any case, it's definitely an oversight that the REFRESH reference
> page fails to address permissions at all. +1 for adding something.
>
> regards, tom lane
Attachments:
0001-docs-note-ownership-requirement-for-refreshing-mater.patch (text/x-patch, 581B) download
diff --git a/doc/src/sgml/ref/refresh_materialized_view.sgml b/doc/src/sgml/ref/refresh_materialized_view.sgml
index 9cf01a25a5..f001a661e1 100644
--- a/doc/src/sgml/ref/refresh_materialized_view.sgml
+++ b/doc/src/sgml/ref/refresh_materialized_view.sgml
@@ -91,6 +91,10 @@ REFRESH MATERIALIZED VIEW [ CONCURRENTLY ] <replaceable class="parameter">name</
<refsect1>
<title>Notes</title>
+ <para>
+ You must own the materialized view in order to use REFRESH MATERIALIZED VIEW.
+ </para>
+
<para>
While the default index for future
<xref linkend="sql-cluster"/>
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-16 22:27 Jonathan S. Katz <jonathan.katz@excoventures.com>
parent: Jonathan S. Katz <jonathan.katz@excoventures.com>
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan S. Katz @ 2018-08-16 22:27 UTC (permalink / raw)
To: Michael Paquier <michael@paquier.xyz>; +Cc: Dian Fay <dian.m.fay@gmail.com>; PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>; Dave Cramer <davecramer@gmail.com>
> On Aug 16, 2018, at 1:05 AM, Jonathan S. Katz <jonathan.katz@excoventures.com> wrote:
>
>>
>> On Aug 15, 2018, at 9:15 PM, Michael Paquier <michael@paquier.xyz> wrote:
>>
>> On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
>>> I played around with this feature a bit and did see this was the case.
>>> Also while playing around I noticed the error message was as such:
>>>
>>> test=> REFRESH MATERIALIZED VIEW blah;
>>> ERROR: must be owner of relation blah
>>>
>>> But it’s not a relation, it’s a materialized view. I attached a patch
>>> that I think should fix this. Kudos to Dave Cramer who was
>>> sitting next to me helping me to locate files and confirm assumptions.
>>
>> A relation may be a materialized view, no? The ACL check happens in
>> RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
>> matview.c).
>
> Comment on the RangeVarCallbackOwnsTable func (abbr):
>
> /*
> * This is intended as a callback for RangeVarGetRelidExtended(). It allows
> * the relation to be locked only if (1) it's a plain table, materialized
> * view, or TOAST table and (2) the current user is the owner (or the
> * superuser). This meets the permission-checking needs of CLUSTER, REINDEX
> * TABLE, and REFRESH MATERIALIZED VIEW; we expose it here so that it can be
> * used by all.
> */
>
> So it’s sharing the permission checking needs amongst all of those commands.
>
> As a user I could be confused if I saw the above error message, esp. because
> the behavior of REFRESH .. is specific to materialized views.
With encouragement from Dave, let me demonstrate what the proposed patch
does to fix the behavior. The steps, running from my “jkatz” user:
CREATE ROLE bar LOGIN;
CREATE TABLE a (x int);
CREATE MATERIALIZED VIEW b AS SELECT * FROM a;
\c - bar
REFRESH MATERIALIZED VIEW b;
ERROR: must be owner of materialized view b
vs. the existing error message which I posted further upthread.
Thanks,
Jonathan
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-17 02:36 Michael Paquier <michael@paquier.xyz>
parent: Dian Fay <dian.m.fay@gmail.com>
0 siblings, 0 replies; 15+ messages in thread
From: Michael Paquier @ 2018-08-17 02:36 UTC (permalink / raw)
To: Dian Fay <dian.m.fay@gmail.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; pgsql-hackers@lists.postgresql.org
On Thu, Aug 16, 2018 at 09:36:29AM -0400, Dian Fay wrote:
> Fair enough! Here's a new version.
Thanks, pushed. Instead of putting the new sentence in the "Notes"
section, I have added it in the "Description" section, which is more
consistent with other commands, like DROP MATERIALIZED VIEW, etc.
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-17 13:08 Dave Cramer <davecramer@gmail.com>
parent: Jonathan S. Katz <jonathan.katz@excoventures.com>
0 siblings, 2 replies; 15+ messages in thread
From: Dave Cramer @ 2018-08-17 13:08 UTC (permalink / raw)
To: Jonathan Katz <jonathan.katz@excoventures.com>; +Cc: Michael Paquier <michael@paquier.xyz>; dian.m.fay@gmail.com, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Dave Cramer
On Thu, 16 Aug 2018 at 18:27, Jonathan S. Katz <
jonathan.katz@excoventures.com> wrote:
>
> On Aug 16, 2018, at 1:05 AM, Jonathan S. Katz <
> jonathan.katz@excoventures.com> wrote:
>
>
> On Aug 15, 2018, at 9:15 PM, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
>
> I played around with this feature a bit and did see this was the case.
> Also while playing around I noticed the error message was as such:
>
> test=> REFRESH MATERIALIZED VIEW blah;
> ERROR: must be owner of relation blah
>
> But it’s not a relation, it’s a materialized view. I attached a patch
> that I think should fix this. Kudos to Dave Cramer who was
> sitting next to me helping me to locate files and confirm assumptions.
>
>
> A relation may be a materialized view, no? The ACL check happens in
> RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
> matview.c).
>
>
> Comment on the RangeVarCallbackOwnsTable func (abbr):
>
> /*
> * This is intended as a callback for RangeVarGetRelidExtended(). It
> allows
> * the relation to be locked only if (1) it's a plain table,
> materialized
> * view, or TOAST table and (2) the current user is the owner (or the
> * superuser). This meets the permission-checking needs of CLUSTER,
> REINDEX
> * TABLE, and REFRESH MATERIALIZED VIEW; we expose it here so that it
> can be
> * used by all.
> */
>
> So it’s sharing the permission checking needs amongst all of those
> commands.
>
> As a user I could be confused if I saw the above error message, esp.
> because
> the behavior of REFRESH .. is specific to materialized views.
>
>
> With encouragement from Dave, let me demonstrate what the proposed patch
> does to fix the behavior. The steps, running from my “jkatz” user:
>
> CREATE ROLE bar LOGIN;
> CREATE TABLE a (x int);
> CREATE MATERIALIZED VIEW b AS SELECT * FROM a;
> \c - bar
> REFRESH MATERIALIZED VIEW b;
> ERROR: must be owner of materialized view b
>
> vs. the existing error message which I posted further upthread.
>
> Thanks,
>
> Jonathan
>
So it seems this patch is being ignored in this thread.
AFAICT, in aclcheck_error() the only way to get to the following sub case
in the case ACLCHECK_NOT_OWNER
case OBJECT_MATVIEW:
msg = gettext_noop("must be owner of materialized view %s");
break;
is if the patch is applied ?
Regards,
Dave
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-17 13:14 Dian Fay <dian.m.fay@gmail.com>
parent: Dave Cramer <davecramer@gmail.com>
1 sibling, 0 replies; 15+ messages in thread
From: Dian Fay @ 2018-08-17 13:14 UTC (permalink / raw)
To: Dave Cramer <davecramer@gmail.com>; Jonathan Katz <jonathan.katz@excoventures.com>; +Cc: Michael Paquier <michael@paquier.xyz>; PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Jonathan's patch seems like a good idea to me from a user POV, but then
I just showed up the other day so I don't really have anything of
substance to add.
On 8/17/18 9:08 AM, Dave Cramer wrote:
>
> Dave Cramer
>
>
> On Thu, 16 Aug 2018 at 18:27, Jonathan S. Katz
> <jonathan.katz@excoventures.com
> <mailto:jonathan.katz@excoventures.com>> wrote:
>
>
>> On Aug 16, 2018, at 1:05 AM, Jonathan S. Katz
>> <jonathan.katz@excoventures.com
>> <mailto:jonathan.katz@excoventures.com>> wrote:
>>
>>>
>>> On Aug 15, 2018, at 9:15 PM, Michael Paquier
>>> <michael@paquier.xyz <mailto:michael@paquier.xyz>> wrote:
>>>
>>> On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
>>>> I played around with this feature a bit and did see this was
>>>> the case.
>>>> Also while playing around I noticed the error message was as such:
>>>>
>>>> test=> REFRESH MATERIALIZED VIEW blah;
>>>> ERROR: must be owner of relation blah
>>>>
>>>> But it’s not a relation, it’s a materialized view. I attached a
>>>> patch
>>>> that I think should fix this. Kudos to Dave Cramer who was
>>>> sitting next to me helping me to locate files and confirm
>>>> assumptions.
>>>
>>> A relation may be a materialized view, no? The ACL check happens in
>>> RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
>>> matview.c).
>>
>> Comment on the RangeVarCallbackOwnsTable func (abbr):
>>
>> /*
>> * This is intended as a callback for
>> RangeVarGetRelidExtended(). It allows
>> * the relation to be locked only if (1) it's a plain table,
>> materialized
>> * view, or TOAST table and (2) the current user is the owner
>> (or the
>> * superuser). This meets the permission-checking needs of
>> CLUSTER, REINDEX
>> * TABLE, and REFRESH MATERIALIZED VIEW; we expose it here so
>> that it can be
>> * used by all.
>> */
>>
>> So it’s sharing the permission checking needs amongst all of
>> those commands.
>>
>> As a user I could be confused if I saw the above error message,
>> esp. because
>> the behavior of REFRESH .. is specific to materialized views.
>
> With encouragement from Dave, let me demonstrate what the proposed
> patch
> does to fix the behavior. The steps, running from my “jkatz” user:
>
> CREATE ROLE bar LOGIN;
> CREATE TABLE a (x int);
> CREATE MATERIALIZED VIEW b AS SELECT * FROM a;
> \c - bar
> REFRESH MATERIALIZED VIEW b;
> ERROR: must be owner of materialized view b
>
> vs. the existing error message which I posted further upthread.
>
> Thanks,
>
> Jonathan
>
>
> So it seems this patch is being ignored in this thread.
> AFAICT, in aclcheck_error() the only way to get to the following sub
> case in the case ACLCHECK_NOT_OWNER
>
> case OBJECT_MATVIEW:
> msg = gettext_noop("must be owner of materialized view %s");
> break;
>
> is if the patch is applied ?
>
> Regards,
>
> Dave
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-17 13:21 Tom Lane <tgl@sss.pgh.pa.us>
parent: Dave Cramer <davecramer@gmail.com>
1 sibling, 1 reply; 15+ messages in thread
From: Tom Lane @ 2018-08-17 13:21 UTC (permalink / raw)
To: Dave Cramer <davecramer@gmail.com>; +Cc: Jonathan Katz <jonathan.katz@excoventures.com>; Michael Paquier <michael@paquier.xyz>; dian.m.fay@gmail.com, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Dave Cramer <davecramer@gmail.com> writes:
> So it seems this patch is being ignored in this thread.
Well, Jonathan did kind of hijack what appears to be a thread about
documentation (with an already-committed fix).
I'd suggest reposting that patch in its own thread and adding it to
the next CF.
regards, tom lane
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-17 21:12 Jonathan S. Katz <jonathan.katz@excoventures.com>
parent: Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan S. Katz @ 2018-08-17 21:12 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Dave Cramer <davecramer@gmail.com>; Michael Paquier <michael@paquier.xyz>; dian.m.fay@gmail.com, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
> On Aug 17, 2018, at 9:21 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Dave Cramer <davecramer@gmail.com> writes:
>> So it seems this patch is being ignored in this thread.
>
> Well, Jonathan did kind of hijack what appears to be a thread about
> documentation (with an already-committed fix).
I apologize if it was interpreted as hijacking, I had brought it up in
my initial reply to Dian, as I’ve seen others do similarly on threads.
> I'd suggest reposting that patch in its own thread and adding it to
> the next CF.
I will go ahead and do this.
Thanks, and thank you for the patch Dian.
Jonathan
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
* Re: docs: note ownership requirement for refreshing materialized views
@ 2018-08-18 08:08 Michael Paquier <michael@paquier.xyz>
parent: Jonathan S. Katz <jonathan.katz@excoventures.com>
0 siblings, 0 replies; 15+ messages in thread
From: Michael Paquier @ 2018-08-18 08:08 UTC (permalink / raw)
To: Jonathan S. Katz <jonathan.katz@excoventures.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; Dave Cramer <davecramer@gmail.com>; dian.m.fay@gmail.com, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
On Fri, Aug 17, 2018 at 05:12:42PM -0400, Jonathan S. Katz wrote:
>> On Aug 17, 2018, at 9:21 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Dave Cramer <davecramer@gmail.com> writes:
>>> So it seems this patch is being ignored in this thread.
>>
>> Well, Jonathan did kind of hijack what appears to be a thread about
>> documentation (with an already-committed fix).
>
> I apologize if it was interpreted as hijacking, I had brought it up in
> my initial reply to Dian, as I’ve seen others do similarly on threads.
Don't worry, we all learn here ;)
I saw your patch, looked at it a couple of seconds and got really
surprised by its logic, moving to something else before lookng back at
it.
>> I'd suggest reposting that patch in its own thread and adding it to
>> the next CF.
>
> I will go ahead and do this.
Thanks for spawning a different thread. That helps in attracting the
correct attention to the correct issues.
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 15+ messages in thread
end of thread, other threads:[~2018-08-18 08:08 UTC | newest]
Thread overview: 15+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2018-08-15 23:46 docs: note ownership requirement for refreshing materialized views Dian Fay <dian.m.fay@gmail.com>
2018-08-16 01:03 ` Michael Paquier <michael@paquier.xyz>
2018-08-16 01:44 ` Dian Fay <dian.m.fay@gmail.com>
2018-08-16 04:07 ` Tom Lane <tgl@sss.pgh.pa.us>
2018-08-16 13:36 ` Dian Fay <dian.m.fay@gmail.com>
2018-08-17 02:36 ` Michael Paquier <michael@paquier.xyz>
2018-08-16 01:06 ` Jonathan S. Katz <jonathan.katz@excoventures.com>
2018-08-16 01:15 ` Michael Paquier <michael@paquier.xyz>
2018-08-16 05:05 ` Jonathan S. Katz <jonathan.katz@excoventures.com>
2018-08-16 22:27 ` Jonathan S. Katz <jonathan.katz@excoventures.com>
2018-08-17 13:08 ` Dave Cramer <davecramer@gmail.com>
2018-08-17 13:14 ` Dian Fay <dian.m.fay@gmail.com>
2018-08-17 13:21 ` Tom Lane <tgl@sss.pgh.pa.us>
2018-08-17 21:12 ` Jonathan S. Katz <jonathan.katz@excoventures.com>
2018-08-18 08:08 ` Michael Paquier <michael@paquier.xyz>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox