Github user zellerh commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1481#discussion_r176177437
--- 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 --
I'm glad you found this. I would recommend to add a big comment saying that
these values are stored in the metadata (COLUMNS.DEFAULT_CLASS) and cannot be
changed (this is in addition to actually assigning the numbers, as Anoop said).
There are only few such enums and it would be good if those were very clearly
marked.
---
