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 Mon, 27 Nov 2017 18:23:30 GMT
Github user zellerh commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/1307#discussion_r153273726
  
    --- 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 --
    
    Instead of adding another constructor, I would suggest to add an optional argument isCaseInSensitive
to the previous constructor.


---

Mime
View raw message