Github user sandhyasun commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1405#discussion_r162448037
--- Diff: core/sql/exp/ExpLOBaccess.h ---
@@ -589,7 +589,7 @@ class ExLobGlobals
{
public :
- ExLobGlobals();
+ ExLobGlobals(NAHeap *lobHeap=NULL);
--- End diff --
Yes this was only done to be able to compile the call in ExpLobProcess.cpp.
Currently this code is unused and is kept around in case we need to offload any
processing to another process. This code, invokes LobGlobals and there is no
heap available. So I had to make it default just for this unused code. WIll
put a comment that it shoudl always pass a heap. In anycase a caller cannot
initialize the LobGlobals without passing in a caller's heap - i.e the only way
to initialize is by calling ExpLOBinterfaceInit or ExpLOBoper::initLobGlobal.
Both require a heap to get passed in.
---
