Date: Sat, 4 Sep 2021 14:55:56 -0500
From: Jaime Casanova <jcasanov@systemguards.com.ec>
To: Paul A Jungwirth <pj@illuminatedcomputing.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: SQL:2011 application time
Message-ID: <20210904195556.GA10553@ahch-to>
References: 
 <CA+renyUApHgSZF9-nd-a0+OPGharLQLO=mDHcY4_qQ0+noCUVg@mail.gmail.com>
 <CA+renyWrT-srf+CiJPA4Uoyx=tmeaCimP_qkJX7w7qSFa8M5uQ@mail.gmail.com>
 <CA+renyW_hn-vuFdW2Atkjrj7E1D13ZEAnuoK9U6YD0p+f2eBAA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="lrZ03NoBR/3+SXJZ"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: 
 <CA+renyW_hn-vuFdW2Atkjrj7E1D13ZEAnuoK9U6YD0p+f2eBAA@mail.gmail.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Archived-At: 
 <https://www.postgresql.org/message-id/20210904195556.GA10553%40ahch-to>
Precedence: bulk


--lrZ03NoBR/3+SXJZ
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Sat, Jul 03, 2021 at 10:46:55AM -0700, Paul A Jungwirth wrote:
> On Fri, Jul 2, 2021 at 2:39 PM Paul A Jungwirth
> <pj@illuminatedcomputing.com> wrote:
> >
> > On Wed, Jun 30, 2021 at 10:39 AM Paul A Jungwirth
> > <pj@illuminatedcomputing.com> wrote:
> > > Here is a set of patches to add SQL:2011 application-time support (aka
> > > valid-time).
> >
> > Here is a small fix to prevent `FOR PORTION OF valid_at FROM MAXVALUE
> > TO foo` or `FROM foo TO MINVALUE`. I rebased on latest master too.
> 
> Here is a patch set that cleans up the catalog docs for pg_period. The
> columns have changed since that was written, and also we use a
> different sgml structure on those pages now. Note pg_period still
> contains a couple essentially-unused columns, perislocal and
> perinhcount. Those are intended for supporting table inheritance, so
> I've left them in.
> 

Hi Paul,

Thanks for working on this. It would be a great improvement.

I wanted to test the patches but:

patch 01: does apply but doesn't compile, attached the compile errors.
patch 04: does not apply clean.

Please fix and resend.

-- 
Jaime Casanova
Director de Servicios Profesionales
SystemGuards - Consultores de PostgreSQL

--lrZ03NoBR/3+SXJZ
Content-Type: text/plain; charset=utf-8
Content-Disposition: attachment;
 filename="v6-0001-Add-PERIODs-compile-errors.log"
Content-Transfer-Encoding: 8bit

parse_utilcmd.c: En la función ‘generateClonedIndexStmt’:
parse_utilcmd.c:1730:2: aviso: ISO C90 prohíbe mezclar declaraciones y código [-Wdeclaration-after-statement]
  Period *p = makeNode(Period);
  ^~~~~~
tablecmds.c: En la función ‘ATPrepCmd’:
tablecmds.c:4637:24: error: tipo incompatible para el argumento 1 de ‘ATSimplePermissions’
    ATSimplePermissions(rel, ATT_TABLE);
                        ^~~
tablecmds.c:403:48: nota: se esperaba ‘AlterTableType’ {también conocido como ‘enum AlterTableType’} pero el argumento es de tipo ‘Relation’ {también conocido como ‘struct RelationData *’}
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
                                 ~~~~~~~~~~~~~~~^~~~~~~
tablecmds.c:308:23: aviso: el paso del argumento 2 de ‘ATSimplePermissions’ crea un puntero desde un entero sin una conversión [-Wint-conversion]
 #define  ATT_TABLE    0x0001
                       ^~~~~~
tablecmds.c:4637:29: nota: en expansión de macro ‘ATT_TABLE’
    ATSimplePermissions(rel, ATT_TABLE);
                             ^~~~~~~~~
tablecmds.c:403:66: nota: se esperaba ‘Relation’ {también conocido como ‘struct RelationData *’} pero el argumento es de tipo ‘int’
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
                                                         ~~~~~~~~~^~~
tablecmds.c:4637:4: error: faltan argumentos para la función ‘ATSimplePermissions’
    ATSimplePermissions(rel, ATT_TABLE);
    ^~~~~~~~~~~~~~~~~~~
tablecmds.c:403:13: nota: se declara aquí
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
             ^~~~~~~~~~~~~~~~~~~
tablecmds.c:4645:24: error: tipo incompatible para el argumento 1 de ‘ATSimplePermissions’
    ATSimplePermissions(rel, ATT_TABLE);
                        ^~~
tablecmds.c:403:48: nota: se esperaba ‘AlterTableType’ {también conocido como ‘enum AlterTableType’} pero el argumento es de tipo ‘Relation’ {también conocido como ‘struct RelationData *’}
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
                                 ~~~~~~~~~~~~~~~^~~~~~~
tablecmds.c:308:23: aviso: el paso del argumento 2 de ‘ATSimplePermissions’ crea un puntero desde un entero sin una conversión [-Wint-conversion]
 #define  ATT_TABLE    0x0001
                       ^~~~~~
tablecmds.c:4645:29: nota: en expansión de macro ‘ATT_TABLE’
    ATSimplePermissions(rel, ATT_TABLE);
                             ^~~~~~~~~
tablecmds.c:403:66: nota: se esperaba ‘Relation’ {también conocido como ‘struct RelationData *’} pero el argumento es de tipo ‘int’
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
                                                         ~~~~~~~~~^~~
tablecmds.c:4645:4: error: faltan argumentos para la función ‘ATSimplePermissions’
    ATSimplePermissions(rel, ATT_TABLE);
    ^~~~~~~~~~~~~~~~~~~
tablecmds.c:403:13: nota: se declara aquí
 static void ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets);
             ^~~~~~~~~~~~~~~~~~~
tablecmds.c: En la función ‘alter_table_type_to_string’:
tablecmds.c:6179:2: aviso: el valor de enumeración ‘AT_AddPeriod’ no se maneja en un switch [-Wswitch]
  switch (cmdtype)
  ^~~~~~
tablecmds.c:6179:2: aviso: el valor de enumeración ‘AT_DropPeriod’ no se maneja en un switch [-Wswitch]
tablecmds.c: En la función ‘ATExecDropPeriod’:
tablecmds.c:7968:23: error: tipo incompatible para el argumento 1 de ‘ATSimplePermissions’
   ATSimplePermissions(rel, ATT_TABLE);
                       ^~~
tablecmds.c:6325:36: nota: se esperaba ‘AlterTableType’ {también conocido como ‘enum AlterTableType’} pero el argumento es de tipo ‘Relation’ {también conocido como ‘struct RelationData *’}
 ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets)
                     ~~~~~~~~~~~~~~~^~~~~~~
tablecmds.c:308:23: aviso: el paso del argumento 2 de ‘ATSimplePermissions’ crea un puntero desde un entero sin una conversión [-Wint-conversion]
 #define  ATT_TABLE    0x0001
                       ^~~~~~
tablecmds.c:7968:28: nota: en expansión de macro ‘ATT_TABLE’
   ATSimplePermissions(rel, ATT_TABLE);
                            ^~~~~~~~~
tablecmds.c:6325:54: nota: se esperaba ‘Relation’ {también conocido como ‘struct RelationData *’} pero el argumento es de tipo ‘int’
 ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets)
                                             ~~~~~~~~~^~~
tablecmds.c:7968:3: error: faltan argumentos para la función ‘ATSimplePermissions’
   ATSimplePermissions(rel, ATT_TABLE);
   ^~~~~~~~~~~~~~~~~~~
tablecmds.c:6325:1: nota: se declara aquí
 ATSimplePermissions(AlterTableType cmdtype, Relation rel, int allowed_targets)
 ^~~~~~~~~~~~~~~~~~~
make[3]: *** [../../../src/Makefile.global:939: tablecmds.o] Error 1
make[2]: *** [common.mk:39: commands-recursive] Error 2
make[2]: *** Se espera a que terminen otras tareas....
make[1]: *** [Makefile:42: all-backend-recurse] Error 2
make: *** [GNUmakefile:11: all-src-recurse] Error 2

--lrZ03NoBR/3+SXJZ--