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

    https://github.com/apache/trafodion/pull/1417#discussion_r165533175
  
    --- 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 --
    
    Could you please comment on how we can attach jdb to say sqlci to debug 
some code in HBaseClient.java or HiveClient.java. I was under the impression 
that jniDebugPort helped with that. I see the two removed instance members have 
now been made static class variables of JOI. I could not catch where they were 
initialized though. Please point me to that code.


---

Reply via email to