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

    
https://github.com/apache/incubator-trafodion/pull/1277#discussion_r147223897
  
    --- Diff: core/sql/common/charinfo.cpp ---
    @@ -693,3 +696,9 @@ Int32 CharInfo::getMaxConvertedLenInBytes(CharSet 
sourceCS,
       return ((sourceLenInBytes/minBytesPerChar(sourceCS)) *
               maxBytesPerChar(targetCS));
     }
    +
    +void CharInfo::initBuiltinCollationDB()
    +{
    +   builtinCollationDB_ = new CollationDB(NULL, mapCOArray, SIZEOF_CO);
    --- End diff --
    
    Is mapCOArray another global object? If it is const it is OK. If not, how 
do we know that it has been constructed already?


---

Reply via email to