Stable version with commented out lines in bind_node_visitor.cpp

Author: Erik Sargent

src/binder/bind_node_visitor.cpp

@@ -259,9 +259,10 @@ void BindNodeVisitor::Visit(expression::TupleValueExpression *expr) {
     //   memory error, but then this prevents the is_not_null flag of a tuple expression from being
     //   populated in some cases (specifically, when the expression's table name is initially empty).
 
-    // if (table_obj == nullptr) {
-    //   BinderContext::GetRegularTableObj(context_, table_name, table_obj, depth);
-    // }
+    //if (table_obj == nullptr) {
+    //  LOG_DEBUG("Extracting regular table object");
+    //  BinderContext::GetRegularTableObj(context_, table_name, table_obj, depth);
+    //}
 
     if (table_obj != nullptr) {
       expr->SetIsNotNull(table_obj->GetColumnCatalogEntry(std::get<2>(col_pos_tuple), false)->IsNotNull());

src/include/expression/tuple_value_expression.h

@@ -98,9 +98,7 @@ class TupleValueExpression : public AbstractExpression {
   }
 
   AbstractExpression *Copy() const override {
-    TupleValueExpression *t = new TupleValueExpression(*this);
-    t->SetIsNotNull(GetIsNotNull());
-    return t;
+    return new TupleValueExpression(*this);
   }
 
   virtual bool operator==(const AbstractExpression &rhs) const override {
@@ -195,7 +193,8 @@ class TupleValueExpression : public AbstractExpression {
         value_idx_(other.value_idx_),
         tuple_idx_(other.tuple_idx_),
         table_name_(other.table_name_),
-        col_name_(other.col_name_) {}
+        col_name_(other.col_name_),
+        is_not_null_(other.is_not_null_) {}
 
   // Bound flag
   bool is_bound_ = false;