Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1405#discussion_r162542021
--- Diff: core/sql/exp/ExpLOBprocess.cpp ---
@@ -575,7 +575,7 @@ Lng32 main(Lng32 argc, char *argv[])
// setup log4cxx
QRLogger::initLog4cxx(QRLogger::QRL_LOB);
// initialize lob globals
- lobGlobals = new ExLobGlobals();
+ lobGlobals = new ExLobGlobals(NULL);
--- End diff --
Thanks for avoiding the default value for the param. But, I think sending
heap as NULL might make this program to fail. Have you tested this program? If
you have tested this program already, then this PR is ready to be merged.
---
