Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/1277#discussion_r147251925
--- Diff: core/sql/common/charinfo.cpp ---
@@ -363,21 +363,22 @@ void CollationInfo::display() const
CollationDB::CollationDB(CollHeap *h)
: CollationDBSupertype(h), heap_(h), refreshNeeded_(TRUE)
{
- if (this == &CharInfo::builtinCollationDB_) return;
+ if (this == CharInfo::builtinCollationDB_) return;
--- End diff --
CharInfo::builtinCollationDB_ is a singleton object. It looks like there
could other CollationDB objects, but it will have all the data in the
builtinCollationDB object too
---
