Github user traflm commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1481#discussion_r176618285
  
    --- Diff: core/sql/regress/seabase/EXPECTED020 ---
    @@ -3945,4 +3945,35 @@ AAAA                       ?            2
     
     --- 1 row(s) selected.
     >>
    +>>obey TEST020(trafodion_2335);
    +>>--create seqence test020_seq;
    +>>create table test020t45(a  largeint not null default unix_timestamp(),
    ++>                        b  char(36) not null default uuid(),
    ++>                        c  varchar(10) default 
to_char(sysdate,'YYYYMMDD'),
    ++>                        --support sequence as default in next check-in
    ++>                        --d  int not null default testi020_seq.nextval,
    ++>                        e  int );
    +
    +--- SQL operation complete.
    +>>insert into test020t45(e) values(1),(2),(3);
    +
    +--- 3 row(s) inserted.
    +>>select count(*) from test020t45 where c = to_char(sysdate,'YYYYMMDD');
    +
    +(EXPR)              
    +--------------------
    +
    +                   3
    +
    +--- 1 row(s) selected.
    +>>--negative tests
    +>>--the function is not variable-free, so should fail
    +>>create table test020t45(a  largeint not null default unix_timestamp(),
    ++>                        b  varchar(10) default 
to_char(test020t45.c,'YYYYMMDD'),
    ++>                        e  int );
    +
    +*** ERROR[1084] An invalid default value was specified for column B.
    +
    +*** ERROR[8822] The statement was not prepared.
    +
    --- End diff --
    
    @selvaganesang By adding new test case for upsert mode 'merge', the missing 
column of unix_timestamp() default value is updated. So still something wrong, 
I need to fix it. Thanks for reminding this.


---

Reply via email to