T_PrivGrantee is left in NodeTag
Hi,
I found that codes about T_PrivGrantee was removed
by the following commit;
http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=31eae6028eca4365e7165f5f33fee1ed0486aee0
but T_PrivGrantee is left in NodeTag in src/include/nodes/nodes.h.
Is it intended?
--
Yugo Nagata <nagata@sraoss.co.jp>
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Hi, thank you for pointing it out.
At Wed, 16 Sep 2015 14:02:30 +0900, Yugo Nagata <nagata@sraoss.co.jp> wrote in <20150916140230.a232426c.nagata@sraoss.co.jp>
I found that codes about T_PrivGrantee was removed
by the following commit;
http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=31eae6028eca4365e7165f5f33fee1ed0486aee0but T_PrivGrantee is left in NodeTag in src/include/nodes/nodes.h.
Is it intended?
I simply forgot to remove it.
The comment for NodeTag says that,
====
* Note that the numbers of the node tags are not contiguous. We left holes
* here so that we can add more tags without changing the existing enum's.
* (Since node tag numbers never exist outside backend memory, there's no
* real harm in renumbering, it just costs a full rebuild ...)
====
However, I think it'd be better to be removed.
Thoughts? The attached patch simply removes it.
--
Kyotaro Horiguchi
NTT Open Source Software Center
Attachments:
0001-Remove-unused-node-type-tag-T_PrivGrantee.patchtext/x-patch; charset=us-asciiDownload
>From c18f99b5eafaa7f3d2867762740bd35133afd3aa Mon Sep 17 00:00:00 2001
From: Kyotaro Horiguchi <horiguchi.kyotaro@lab.ntt.co.jp>
Date: Wed, 16 Sep 2015 16:06:09 +0900
Subject: [PATCH] Remove unused node type tag T_PrivGrantee.
T_PrivGrantee became useless by the commit
31eae6028eca4365e7165f5f33fee1ed0486aee0. This commit renumbers some
of node type tags.
---
src/include/nodes/nodes.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/include/nodes/nodes.h b/src/include/nodes/nodes.h
index 748e434..274480e 100644
--- a/src/include/nodes/nodes.h
+++ b/src/include/nodes/nodes.h
@@ -413,7 +413,6 @@ typedef enum NodeTag
T_SortGroupClause,
T_GroupingSet,
T_WindowClause,
- T_PrivGrantee,
T_FuncWithArgs,
T_AccessPriv,
T_CreateOpClassItem,
--
1.8.3.1
Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> writes:
At Wed, 16 Sep 2015 14:02:30 +0900, Yugo Nagata <nagata@sraoss.co.jp> wrote in <20150916140230.a232426c.nagata@sraoss.co.jp>
but T_PrivGrantee is left in NodeTag in src/include/nodes/nodes.h.
Is it intended?
Thoughts? The attached patch simply removes it.
Yeah, we can just take it out I think, even in the 9.5 branch. If we
were past 9.5beta1 then I'd be worried about creating an ABI break for
third-party extensions, but we're still making ABI changes for other
things.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers