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

    
https://github.com/apache/incubator-trafodion/pull/1277#discussion_r147250524
  
    --- 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 --
    
    Yes. C++ prologue ensures that it is initialized correctly. In master 
process, it will be initialized before dlOpen and in other processes it should 
be done before main is called.


---

Reply via email to