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

    https://github.com/apache/trafodion/pull/1481#discussion_r175964689
  
    --- Diff: core/sql/common/ComSmallDefs.h ---
    @@ -609,47 +609,59 @@ enum ComColumnClass { COM_UNKNOWN_CLASS
     #define COM_ALTERED_USER_COLUMN_LIT         "C "
     
     enum ComColumnDefaultClass { COM_CURRENT_DEFAULT
    +                           , COM_CURRENT_UT_DEFAULT
    --- End diff --
    
    @robertamarton Could you help to provide some comments here?
    @selvaganesang  I think new values will NOT cause problems, the MD table 
structure is not changed. I don't find any checking code relying on values in 
this column. But Roberta should have more insight here.


---

Reply via email to