Changes from code review

Author: mhaseeb123

velox/exec/tests/utils/PlanBuilder.cpp

@@ -299,7 +299,7 @@ core::PlanNodePtr PlanBuilder::TableWriterBuilder::build(core::PlanNodeId id) {
       outputType,
       outputType->names(),
       aggregationNode,
-      insertHandle,
+      insertHandle_,
       false,
       TableWriteTraits::outputType(aggregationNode),
       connector::CommitStrategy::kNoCommit,

velox/exec/tests/utils/PlanBuilder.h

@@ -431,7 +431,7 @@ class PlanBuilder {
     std::string outputDirectoryPath_;
     std::string outputFileName_;
     std::string connectorId_{kHiveDefaultConnectorId};
-    shared_ptr<core::InsertTableHandle> insertHandle_;
+    std::shared_ptr<core::InsertTableHandle> insertHandle_;
 
     std::vector<std::string> partitionBy_;
     int32_t bucketCount_{0};