Github user selvaganesang commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1417#discussion_r167473755
--- Diff: core/sql/executor/ExExeUtilGet.cpp ---
@@ -3521,13 +3521,9 @@
ExExeUtilGetHbaseObjectsTcb::ExExeUtilGetHbaseObjectsTcb(
ex_globals * glob)
: ExExeUtilGetMetadataInfoTcb( exe_util_tdb, glob)
{
- int jniDebugPort = 0;
- int jniDebugTimeout = 0;
ehi_ = ExpHbaseInterface::newInstance(glob->getDefaultHeap(),
(char*)exe_util_tdb.server(),
- (char*)exe_util_tdb.zkPort(),
- jniDebugPort,
- jniDebugTimeout);
+ (char*)exe_util_tdb.zkPort());
--- End diff --
DebugPort needs to be initialized as part of JVM invocation. See
JavaObjectInterface:;createJVM. This debugPort is never used. Because it is per
process, it needs be be made static
---
