trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robertamarton <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement
Date Thu, 02 Nov 2017 20:45:27 GMT
Github user robertamarton commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654612
  
    --- Diff: core/sql/sqlcomp/CmpSeabaseDDLtable.cpp ---
    @@ -12940,6 +12940,142 @@ TrafDesc *CmpSeabaseDDL::getSeabaseRoutineDescInternal(const
NAString &catName,
     }
     
     
    +short CmpSeabaseDDL::getSeabaseObjectComment(Int64 object_uid, 
    +                                                    enum ComObjectType object_type, 
    +                                                    ComTdbVirtObjCommentInfo * &
comment_info)
    +{
    +  Lng32 retcode = 0;
    +  Lng32 cliRC = 0;
    +
    +  char query[4000];
    +
    +  comment_info = NULL;
    +
    +
    +  ExeCliInterface cliInterface(STMTHEAP, 
    +                               NULL, NULL, 
    +                               CmpCommon::context()->sqlSession()->getParentQid());
    +
    +  //get object comment
    +  str_sprintf(query, "select comment from %s.\"%s\".%s where object_uid = %ld and object_type
= '%s' and comment <> '' ;",
    --- End diff --
    
    Recently all str_sprintf's were changed to sprintf's.  You might want to change this code
also.


---

Mime
View raw message