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

    https://github.com/apache/trafodion/pull/1658#discussion_r206707222
  
    --- Diff: core/sql/optimizer/ItemExpr.cpp ---
    @@ -11971,6 +11971,43 @@ Exists::copyTopNode(ItemExpr *derivedNode, 
CollHeap* outHeap)
     // <aviv> END
     
     
    +// --------------------------------------------------------------
    +// member functions for Overlaps operator
    +// --------------------------------------------------------------
    +ItemExpr * Overlaps::copyTopNode(ItemExpr *derivedNode, CollHeap* outHeap)
    +{
    +  ItemExpr *result;
    +  if (derivedNode == NULL)
    +    result = new (outHeap) Overlaps(child(0), child(1), child(2), 
child(3));
    +  else
    +    result - derivedNode;
    +
    +  return BuiltinFunction::copyTopNode(result, outHeap);
    --- End diff --
    
    This should call CacheableBuiltinFunction::copyTopNode().


---

Reply via email to