[GitHub] trafodion pull request #1417: [TRAFODION-2917] Refactor Trafodion implementa...

[eowhadi]

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

    https://github.com/apache/trafodion/pull/1417#discussion_r164189696
  
    --- Diff: core/sql/executor/ExHbaseAccess.cpp ---
    @@ -502,6 +506,8 @@ void ExHbaseAccessTcb::freeResources()
          NADELETEBASIC(directRowBuffer_, getHeap());
       if (colVal_.val != NULL)
          NADELETEBASIC(colVal_.val, getHeap());
    +  if (hdfsClient_ != NULL) 
    +     NADELETE(hdfsClient_, HdfsClient, getHeap());
     }
    --- End diff --
    
    I am wondering if there is a need to delete the new introduces 
loggingFileName_ here?


---