ECPG: pg_type.h file is not synced
Hello,
I am looking at ECPG source code. In the "ecpg/ecpglib/pg_type.h" file I
have seen following comment:
** keep this in sync with src/include/catalog/pg_type.h*
But I think the "ecpg/ecpglib/pg_type.h" file is currently not synced
with the above file.
I have added the remaining types in the attached patch.
I would like to know whether we can add remaining types in the
"ecpg/ecpglib/pg_type.h" file or not?
Regards,
Vinayak Pokale
NTT Open Source Software Center
Attachments:
ECPG_pg_type_h.patchbinary/octet-stream; name=ECPG_pg_type_h.patchDownload
diff --git a/src/interfaces/ecpg/ecpglib/pg_type.h b/src/interfaces/ecpg/ecpglib/pg_type.h
index c86af25..467824e 100644
--- a/src/interfaces/ecpg/ecpglib/pg_type.h
+++ b/src/interfaces/ecpg/ecpglib/pg_type.h
@@ -57,5 +57,23 @@
#define ZPBITOID 1560
#define VARBITOID 1562
#define NUMERICOID 1700
+#define REFCURSOROID 1790
+#define REGPROCEDUREOID 2202
+#define REGOPEROID 2203
+#define REGOPERATOROID 2204
+#define REGCLASSOID 2205
+#define REGTYPEOID 2206
+#define REGROLEOID 4096
+#define REGNAMESPACEOID 4089
+#define REGTYPEARRAYOID 2211
+#define UUIDOID 2950
+#define LSNOID 3220
+#define TSVECTOROID 3614
+#define GTSVECTOROID 3642
+#define TSQUERYOID 3615
+#define REGCONFIGOID 3734
+#define REGDICTIONARYOID 3769
+#define JSONBOID 3802
+#define INT4RANGEOID 3904
#endif /* PG_TYPE_H */
Hi,
I am looking at ECPG source code. In the "ecpg/ecpglib/pg_type.h" file I
have seen following comment:** keep this in sync with src/include/catalog/pg_type.h*
But I think the "ecpg/ecpglib/pg_type.h" file is currently not synced
with the above file.I have added the remaining types in the attached patch.
I would like to know whether we can add remaining types in the
"ecpg/ecpglib/pg_type.h" file or not?
It's not too big a deal as neither of the new OIDs is used in ecpg, but
still, the file should be up to date.
Thanks for the patch, committed.
Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Meskes at (Debian|Postgresql) dot Org
Jabber: michael at xmpp dot meskes dot org
VfL Borussia! Força Barça! Go SF 49ers! Use Debian GNU/Linux, PostgreSQL
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
On Tue, May 23, 2017 at 10:25:13AM +0200, Michael Meskes wrote:
Hi,
I am looking at ECPG source code. In the "ecpg/ecpglib/pg_type.h" file I
have seen following comment:** keep this in sync with src/include/catalog/pg_type.h*
But I think the "ecpg/ecpglib/pg_type.h" file is currently not synced
with the above file.I have added the remaining types in the attached patch.
I would like to know whether we can add remaining types in the
"ecpg/ecpglib/pg_type.h" file or not?It's not too big a deal as neither of the new OIDs is used in ecpg, but
still, the file should be up to date.Thanks for the patch, committed.
Should we add some Makefile magic to make sure they stay in sync?
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Bruce Momjian <bruce@momjian.us> writes:
But I think the "ecpg/ecpglib/pg_type.h" file is currently not synced
with the above file.
Should we add some Makefile magic to make sure they stay in sync?
Not so much that as teach genbki.pl to generate this file too.
I think that might be a good idea, but personally I'd leave it until
after the genbki rewrite we keep talking about. This isn't important
enough to make that rewrite have an even higher hurdle to clear.
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
Bruce Momjian <bruce@momjian.us> writes:
But I think the "ecpg/ecpglib/pg_type.h" file is currently not
synced
with the above file.Should we add some Makefile magic to make sure they stay in sync?
Not so much that as teach genbki.pl to generate this file too.
I think that might be a good idea, but personally I'd leave it until
after the genbki rewrite we keep talking about. This isn't important
enough to make that rewrite have an even higher hurdle to clear.regards, tom lane
Agreed. We will only run into problems with ecpg if some of the
existing oids change. Newly added ones will not be referenced unless
ecpg receives changes, too.
Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Meskes at (Debian|Postgresql) dot Org
Jabber: michael at xmpp dot meskes dot org
VfL Borussia! Força Barça! SF 49ers! Use Debian GNU/Linux, PostgreSQL
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers