trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zellerh <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #1307: [TRAFODION-2791] 'Not casespecific' ...
Date Tue, 28 Nov 2017 17:07:53 GMT
Github user zellerh commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/1307#discussion_r153559391
  
    --- Diff: core/sql/optimizer/ItemColRef.h ---
    @@ -350,6 +350,14 @@ class ConstValue : public ItemExpr
                NAMemory * outHeap = CmpCommon::statementHeap()
             );
     
    +  ConstValue(const NAString& strval,
    +           NABoolean isCaseInSensitive,
    +           enum CharInfo::CharSet charSet=CharInfo::DefaultCharSet,
    +           enum CharInfo::Collation collation=CharInfo::DefaultCollation,
    +           enum CharInfo::Coercibility coercibility=CharInfo::COERCIBLE,
    +           NAMemory * outHeap = CmpCommon::statementHeap()
    +        );
    --- End diff --
    
    Yes, you need to add it as the last argument of the existing constructors.


---

Mime
View raw message