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

    https://github.com/apache/trafodion/pull/1578#discussion_r190645661
  
    --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommentOn.cpp ---
    @@ -229,6 +229,16 @@ void  
CmpSeabaseDDL::doSeabaseCommentOn(StmtDDLCommentOn   *commentOnNode,
       enum ComObjectType enMDObjType = COM_UNKNOWN_OBJECT;
     
       ComObjectName objectName(commentOnNode->getObjectName());
    +  //we can not comment on native hive table.
    +  if (str_cmp(toUpper(currCatName.data()), "HIVE", 4) == 0)
    --- End diff --
    
    Please use the literal HIVE_SYSTEM_CATALOG (see common/ComSmallDefs.h) 
instead of hard-coding "HIVE".


---

Reply via email to