[GitHub] incubator-trafodion pull request #1355: [TRAFODION-2853] Memory leak of ComD...

2017-12-18 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1355 [TRAFODION-2853] Memory leak of ComDiagsArea in Context CLI entry points to LOB and Sequence generator didn't deallocate the local diags area. You can merge this pull

[GitHub] incubator-trafodion pull request #1351: [TRAFODION-2733] BMO quota changes

2017-12-15 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1351 [TRAFODION-2733] BMO quota changes Fix for hive/TEST009 regression failure with commit 375eeeab7a You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-trafodion pull request #1341: [TRAFODION-2733] BMO quota changes

2017-12-13 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1341 [TRAFODION-2733] BMO quota changes To ensure BMO quota assignment works even when the estimates are way off. Introduced a CQD BMO_MEMORY_ESTIMATE_OUTLIER_FACTOR. When

[GitHub] incubator-trafodion pull request #1311: [TRAFODION-2818] For Server 2008, fu...

2017-12-08 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1311#discussion_r155901708 --- Diff: win-odbc64/odbcclient/drvr35/sqltocconv.cpp --- @@ -36,6 +36,34 @@ #define MAXCHARLEN 32768 //32K

[GitHub] incubator-trafodion pull request #1332: Fixes for [TRAFODION-2836] and [TRAF...

2017-12-08 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1332 Fixes for [TRAFODION-2836] and [TRAFODION-2420] You can merge this pull request into a Git repository by running: $ git pull https://github.com/selvaganesang/incubator

[GitHub] incubator-trafodion pull request #1311: [TRAFODION-2818] For Server 2008, fu...

2017-11-29 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1311#discussion_r153837812 --- Diff: win-odbc64/odbcclient/drvr35/sqltocconv.cpp --- @@ -36,6 +36,34 @@ #define MAXCHARLEN 32768 //32K

[GitHub] incubator-trafodion pull request #1310: [TRAFODION-2816] ODBC data convert f...

2017-11-29 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1310#discussion_r153831339 --- Diff: core/conn/unixodbc/odbc/odbcclient/unixcli/cli/drvrglobal.cpp --- @@ -1092,6 +1092,106 @@ bool ctoi64(char* string, __int64

[GitHub] incubator-trafodion pull request #1314: [TRAFODION-2821] Trafodion core code...

2017-11-28 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1314#discussion_r153585118 --- Diff: core/sql/executor/SequenceFileReader.cpp --- @@ -189,9 +192,10 @@ SFR_RetCode SequenceFileReader::open(const char* path

[GitHub] incubator-trafodion pull request #1314: [TRAFODION-2821] Trafodion core code...

2017-11-28 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1314#discussion_r153585050 --- Diff: core/sql/executor/OrcFileReader.cpp --- @@ -236,58 +314,71 @@ OFR_RetCode OrcFileReader::fetchNextRow(char * buffer, long

[GitHub] incubator-trafodion pull request #1314: [TRAFODION-2821] Trafodion core code...

2017-11-28 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1314#discussion_r153584842 --- Diff: core/sql/executor/OrcFileReader.cpp --- @@ -122,24 +150,28 @@ OFR_RetCode OrcFileReader::init() OFR_RetCode OrcFileReader

[GitHub] incubator-trafodion pull request #1314: [TRAFODION-2821] Trafodion core code...

2017-11-28 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1314 [TRAFODION-2821] Trafodion core code base needs to be thread safe The method Ids of the different Java classes are now initialized in a thread safe manner in JNI layer

[GitHub] incubator-trafodion pull request #1294: [TRAFODION-2801] select count(*) wit...

2017-11-09 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1294 [TRAFODION-2801] select count(*) with cqd hbase_coprocessors 'off' and cqd traf_table_snapshot_scan 'latest' fail Changed to pass two more arrays references

[GitHub] incubator-trafodion pull request #1287: [TRAFODION-2785] mxosrvr dumps core ...

2017-10-31 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1287 [TRAFODION-2785] mxosrvr dumps core generated when loading data Expected file change for the failures observed in daily build due to fix for the above problem You can merge

[GitHub] incubator-trafodion pull request #1280: [TRAFODION-2785] mxosrvr dumps core ...

2017-10-27 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1280 [TRAFODION-2785] mxosrvr dumps core generated when loading data from … …Oracle to Trafodion at times When there is an error while loading the data, the hexadecimal

[GitHub] incubator-trafodion pull request #1279: [TRAFODION-2420] RMS Enhancements

2017-10-27 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1279 [TRAFODION-2420] RMS Enhancements Refactored the ESP memory management so that all the SQL memory both heap and space objects are accounted correctly in RMS

[GitHub] incubator-trafodion pull request #1277: [TRAFODION-2783] jdbc_test_cdh fails...

2017-10-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1277#discussion_r147251925 --- Diff: core/sql/common/charinfo.cpp --- @@ -363,21 +363,22 @@ void CollationInfo::display() const CollationDB::CollationDB

[GitHub] incubator-trafodion pull request #1277: [TRAFODION-2783] jdbc_test_cdh fails...

2017-10-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1277#discussion_r147250690 --- Diff: core/sql/common/charinfo.cpp --- @@ -529,8 +530,10 @@ static const CollationInfo mapCOArray[] = { CollationInfo(NULL

[GitHub] incubator-trafodion pull request #1277: [TRAFODION-2783] jdbc_test_cdh fails...

2017-10-25 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1277 [TRAFODION-2783] jdbc_test_cdh fails at times with type 2 JDBC driver… … dumping core In case of Type 2 JDBC driver, the Trafodion SQL engine is a library

[GitHub] incubator-trafodion pull request #1273: [TRAFODION-2780] The mxosrvr dumps c...

2017-10-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1273#discussion_r146354070 --- Diff: core/conn/odbc/src/odbc/nsksrvr/SrvrConnect.cpp --- @@ -1190,10 +1176,6 @@ ImplInit ( srvrGlobal->receiveTh

[GitHub] incubator-trafodion pull request #1272: [TRAFODION-2773] When the timeout is...

2017-10-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1272#discussion_r146286486 --- Diff: core/conn/odb/src/odb.c --- @@ -4156,18 +4156,17 @@ static void sigcatch(int sig) exit ( EX_SIGNAL ); #else

[GitHub] incubator-trafodion pull request #1264: [TRAFODION-2772] - retrieve a value ...

2017-10-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1264#discussion_r146280638 --- Diff: core/sql/exp/exp_function.cpp --- @@ -6503,8 +6503,19 @@ ex_expr::exp_return_type ex_function_json_object_field_text::eval

[GitHub] incubator-trafodion pull request #1273: [TRAFODION-2780] The mxosrvr dumps c...

2017-10-20 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1273 [TRAFODION-2780] The mxosrvr dumps core when connection idle timer ex… …pires at times This core is accompanied by mxssmp process core. Got rid of both

[GitHub] incubator-trafodion pull request #1250: [TRAFODION-2758] Sort operator that ...

2017-10-02 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1250#discussion_r142212229 --- Diff: core/sql/optimizer/RelExpr.cpp --- @@ -1078,7 +1078,33 @@ void RelExpr::pushDownGenericUpdateRootOutputs( const ValueIdSet

[GitHub] incubator-trafodion pull request #1252: [TRAFODION-2754] Get statistics core...

2017-10-01 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1252 [TRAFODION-2754] Get statistics cores sqlci or mxosrvr at str_sprintf() Fix for executor/TEST130 failure in daily build You can merge this pull request into a Git repository

[GitHub] incubator-trafodion pull request #1241: [TRAFODION-2750] Using function strt...

2017-09-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1241#discussion_r141240746 --- Diff: core/conn/unixodbc/odbc/odbcclient/unixcli/cli/ctosqlconv.cpp --- @@ -3545,7 +3545,7 @@ unsigned long ODBC

[GitHub] incubator-trafodion pull request #1245: [TRAFODION-2754] Get statistics core...

2017-09-25 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1245 [TRAFODION-2754] Get statistics cores sqlci or mxosrvr at str_sprintf() str_sprintf is Trafodion specific formatting function. Replaced str_sprintf with sprint that provides

[GitHub] incubator-trafodion pull request #1241: [TRAFODION-2750] Using function strt...

2017-09-21 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1241#discussion_r140350755 --- Diff: core/conn/unixodbc/odbc/odbcclient/unixcli/cli/ctosqlconv.cpp --- @@ -3545,7 +3545,7 @@ unsigned long ODBC

[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...

2017-09-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r139488348 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/ListenerWorker.java --- @@ -86,39 +86,43 @@ public void run

[GitHub] incubator-trafodion pull request #1232: [TRAFODION-2738] Rowset buffer size ...

2017-09-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1232#discussion_r139270056 --- Diff: core/sql/sqlcomp/nadefaults.cpp --- @@ -1336,6 +1336,8 @@ SDDkwd__(EXE_DIAGNOSTIC_EVENTS, &quo

[GitHub] incubator-trafodion pull request #1220: [TRAFODION-2725] SQL types are REAL,...

2017-09-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1220#discussion_r139188923 --- Diff: win-odbc64/odbcclient/drvr35/drvrglobal.cpp --- @@ -865,77 +865,18 @@ bool use_gcvt(double number, char* string, short size

[GitHub] incubator-trafodion pull request #1231: [Trafodion 2739] RMS semaphore handl...

2017-09-13 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1231#discussion_r138777511 --- Diff: core/sql/cli/Cli.cpp --- @@ -8325,19 +8325,15 @@ Lng32 SQLCLI_RegisterQuery(CliGlobals *cliGlobals, StatsGlobals

[GitHub] incubator-trafodion pull request #1231: [Trafodion 2733] RMS semaphore handl...

2017-09-13 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1231 [Trafodion 2733] RMS semaphore handling need to log the error for easy resolution of the issue RMS semaphore handling had the following: 1) Passed in parameters

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-13 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138711021 --- Diff: core/sql/executor/ex_sort.cpp --- @@ -364,12 +365,15 @@ ExSortTcb::ExSortTcb(const ExSortTdb & sort

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-13 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138695477 --- Diff: core/sql/cli/Context.h --- @@ -505,6 +519,8 @@ class ContextCli : public ExGod { // It points to a list of 'black box

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-13 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138634994 --- Diff: core/sql/generator/GenProbeCache.cpp --- @@ -470,30 +463,38 @@ CostScalar ProbeCache::getEstimatedRunTimeMemoryUsage

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138499497 --- Diff: core/sql/generator/GenProbeCache.cpp --- @@ -470,30 +463,38 @@ CostScalar ProbeCache::getEstimatedRunTimeMemoryUsage

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138499177 --- Diff: core/sql/generator/GenProbeCache.cpp --- @@ -470,30 +463,38 @@ CostScalar ProbeCache::getEstimatedRunTimeMemoryUsage

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138498956 --- Diff: core/sql/optimizer/RelExpr.cpp --- @@ -1988,77 +1988,62 @@ NABoolean RelExpr::containsNode(OperatorTypeEnum nodeType

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138498601 --- Diff: core/sql/optimizer/RelExpr.cpp --- @@ -1988,77 +1988,62 @@ NABoolean RelExpr::containsNode(OperatorTypeEnum nodeType

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138498212 --- Diff: core/sql/generator/GenPreCode.cpp --- @@ -2154,34 +2154,10 @@ RelExpr * RelRoot::preCodeGen(Generator * generator

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138498166 --- Diff: core/sql/executor/ex_exe_stmt_globals.h --- @@ -307,18 +307,38 @@ class ExExeStmtGlobals : public ex_globals

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138492138 --- Diff: core/sql/cli/Globals.cpp --- @@ -1099,6 +1100,52 @@ void CliGlobals::deleteContexts() } #endif // _DEBUG

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138491738 --- Diff: core/sql/generator/GenRelJoin.cpp --- @@ -1784,36 +1784,44 @@ short HashJoin::codeGen(Generator * generator) { UInt16

[GitHub] incubator-trafodion pull request #1228: [TRAFODION-2733] Provide an improved...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138487244 --- Diff: core/sql/comexe/ComTdbSort.h --- @@ -201,11 +201,15 @@ class ComTdbSort : public ComTdb Int32 sortPartialKeyLen_

[GitHub] incubator-trafodion pull request #1225: [TRAFODION-2704]close unused statmen...

2017-09-12 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1225#discussion_r138397172 --- Diff: core/conn/odbc/src/odbc/nsksrvrcore/srvrothers.cpp --- @@ -6489,6 +6506,31 @@ odbc_SQLSrvr_ExtractLob_sme_

[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...

2017-09-06 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r137416427 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/ListenerWorker.java --- @@ -86,39 +86,43 @@ public void run

[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...

2017-09-01 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r136582435 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/ListenerWorker.java --- @@ -86,39 +86,43 @@ public void run

[GitHub] incubator-trafodion pull request #1219: [TRAFODION-2727] Memory leak in the ...

2017-08-29 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1219 [TRAFODION-2727] Memory leak in the compiler part of the code in Trafodion Compiler objects were refactored to use the heap infrastructure to ensure that it is accounted

[GitHub] incubator-trafodion pull request #1213: implement isValid function

2017-08-24 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1213#discussion_r135030383 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4Connection.java --- @@ -1876,9 +1876,57 @@ public SQLXML

[GitHub] incubator-trafodion pull request #1213: implement isValid function

2017-08-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1213#discussion_r134927206 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4Connection.java --- @@ -1876,9 +1876,57 @@ public SQLXML

[GitHub] incubator-trafodion pull request #1216: TRAFODION-2724

2017-08-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1216#discussion_r134892384 --- Diff: core/sql/exp/ExpLOBaccess.cpp --- @@ -1970,7 +1970,14 @@ Ex_Lob_Error ExLob::readCursorData(char *tgt, Int64 tgtSize

[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...

2017-08-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r134071493 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/RequestUnknown.java --- @@ -76,7 +76,11 @@ ClientData processRequest

[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...

2017-08-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r134071398 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/ListenerWorker.java --- @@ -86,39 +86,43 @@ public void run

[GitHub] incubator-trafodion pull request #1211: TRAFODION-2722 Change purgedata to u...

2017-08-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1211#discussion_r134067996 --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java --- @@ -648,6 +648,8 @@ public boolean truncate(String tblName

[GitHub] incubator-trafodion pull request #1211: TRAFODION-2722 Change purgedata to u...

2017-08-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1211#discussion_r134066270 --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java --- @@ -648,6 +648,8 @@ public boolean truncate(String tblName

[GitHub] incubator-trafodion pull request #1206: [TRAFODION-2617] Optionally push est...

2017-08-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1206#discussion_r133314557 --- Diff: core/sql/sqlcomp/nadefaults.cpp --- @@ -1759,6 +1763,7 @@ SDDkwd__(EXE_DIAGNOSTIC_EVENTS, &quo

[GitHub] incubator-trafodion pull request #1206: [TRAFODION-2617] Optionally push est...

2017-08-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1206#discussion_r133307227 --- Diff: core/sql/sqlcomp/nadefaults.cpp --- @@ -1759,6 +1763,7 @@ SDDkwd__(EXE_DIAGNOSTIC_EVENTS, &quo

[GitHub] incubator-trafodion pull request #1206: [TRAFODION-2617] Optionally push est...

2017-08-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1206#discussion_r133294784 --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java --- @@ -1395,6 +1422,169 @@ private boolean estimateRowCountBody

[GitHub] incubator-trafodion pull request #1206: [TRAFODION-2617] Optionally push est...

2017-08-15 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1206#discussion_r133294368 --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java --- @@ -1395,6 +1422,169 @@ private boolean estimateRowCountBody

[GitHub] incubator-trafodion pull request #1204: [TRAFODION-2217] Error 8026 is shown...

2017-08-13 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1204 [TRAFODION-2217] Error 8026 is shown at times when the query is cance… …lled. control query cancel for qid fails with error *** ERROR[8026] Server declined

[GitHub] incubator-trafodion pull request #1191: [TRAFODION-2698] Ensure RMS can be d...

2017-07-28 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1191 [TRAFODION-2698] Ensure RMS can be disabled properly You can merge this pull request into a Git repository by running: $ git pull https://github.com/selvaganesang

[GitHub] incubator-trafodion pull request #1189: [TRAFODION-2696] control query cance...

2017-07-28 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1189 [TRAFODION-2696] control query cancel qid fails with error 8031 somet… …imes With this change, ssmp will not be stopped when the query is stuck at the time

[GitHub] incubator-trafodion pull request #1188: [TRAFODION-2695] SSMP process ($ZSMx...

2017-07-25 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1188 [TRAFODION-2695] SSMP process ($ZSMxxx) sees too many opens from the … …master process Mxosrvr/Any master process opens a connection to mxssmp for the following

[GitHub] incubator-trafodion pull request #1186: [TRAFODION-2596] Improve the log4j a...

2017-07-18 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1186 [TRAFODION-2596] Improve the log4j and log4cxx infrastructure in Traf… …odion Missed out changes in the configuration file in PR 1178 You can merge this pull

[GitHub] incubator-trafodion pull request #1175: TRAFODION-2686 fix setCurSorName doe...

2017-07-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1175#discussion_r128053948 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4Statement.java --- @@ -1247,7 +1247,7 @@ void

[GitHub] incubator-trafodion pull request #1170: [TRAFODION-2628] SQL engine work for...

2017-07-17 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1170#discussion_r127814010 --- Diff: core/sql/optimizer/NAClusterInfo.cpp --- @@ -350,14 +244,8 @@ NAClusterInfo::NAClusterInfo(CollHeap * heap

[GitHub] incubator-trafodion pull request #1170: [TRAFODION-2628] SQL engine work for...

2017-07-17 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1170#discussion_r127792169 --- Diff: core/sql/optimizer/NAClusterInfo.cpp --- @@ -350,14 +244,8 @@ NAClusterInfo::NAClusterInfo(CollHeap * heap

[GitHub] incubator-trafodion pull request #1170: [TRAFODION-2628] SQL engine work for...

2017-07-17 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1170#discussion_r127782107 --- Diff: core/sql/cli/Globals.cpp --- @@ -187,18 +187,9 @@ void CliGlobals::init( NABoolean espProcess

[GitHub] incubator-trafodion pull request #1180: [TRAFODION-2685] int type doesnot ch...

2017-07-17 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1180#discussion_r127725164 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4ResultSet.java --- @@ -1372,7 +1372,7 @@ public long getLong(int

[GitHub] incubator-trafodion pull request #1175: TRAFODION-2686 fix setCurSorName doe...

2017-07-17 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1175#discussion_r127723191 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4Statement.java --- @@ -1247,7 +1247,7 @@ void

[GitHub] incubator-trafodion pull request #1184: [TRAFODION-2420] RMS enhancements

2017-07-16 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1184 [TRAFODION-2420] RMS enhancements Added four new counters to provide information about multiple executes of a prepared statement. No. of times executed Min

[GitHub] incubator-trafodion pull request #1182: [TRAFODION-2688] Need a way to find ...

2017-07-14 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1182 [TRAFODION-2688] Need a way to find queries that are stuck in compila… …tion via RMS When a query is stuck in compilation, the mxosrvr remains in connected state

[GitHub] incubator-trafodion pull request #1178: [TRAFODION-2596] Improve the log4j a...

2017-07-13 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1178 [TRAFODION-2596] Improve the log4j and log4cxx infrastructure in Traf… …odion Removed setting the root logger to be the appender logger in most

[GitHub] incubator-trafodion pull request #1172: [TRAFODION-2681] Repeated execution ...

2017-07-10 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1172 [TRAFODION-2681] Repeated execution of prepared SQL select statement … …causes memory leak This leak can be seen with the mdam plans when it attempts to do multiple

[GitHub] incubator-trafodion pull request #1143: [TRAFODION-2654] Change the location...

2017-06-27 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1143#discussion_r124345414 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/TrxRegionEndpoint.java.tmpl

[GitHub] incubator-trafodion pull request #1140: [TRAFODION-2663] Simplify HBase conf...

2017-06-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1140#discussion_r124106179 --- Diff: install/python-installer/scripts/hadoop_mods.py --- @@ -122,10 +122,6 @@ def mod(self

[GitHub] incubator-trafodion pull request #1144: TRAFODION-2668 one mxosrvr cant star...

2017-06-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1144#discussion_r124095950 --- Diff: dcs/src/main/java/org/trafodion/dcs/server/ServerManager.java --- @@ -414,32 +431,40 @@ public Boolean call() throws Exception

[GitHub] incubator-trafodion pull request #1144: TRAFODION-2668 one mxosrvr cant star...

2017-06-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1144#discussion_r124078860 --- Diff: dcs/src/main/java/org/trafodion/dcs/server/ServerManager.java --- @@ -127,12 +125,17 @@ public ServerMonitor(int childInstance

[GitHub] incubator-trafodion pull request #1141: [TRAFODION-2637] Library management ...

2017-06-26 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1141#discussion_r124065260 --- Diff: core/sql/udrserv/udrserv.cpp --- @@ -230,35 +230,6 @@ void InitializeJavaOptionsFromEnvironment(LmJavaOptions

[GitHub] incubator-trafodion pull request #1113: [TRAFODION-2636] Memory leak in meta...

2017-06-07 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1113#discussion_r120700381 --- Diff: core/sql/optimizer/ControlDB.cpp --- @@ -108,7 +108,12 @@ ControlDB::~ControlDB() void ControlDB::setRequiredShape

[GitHub] incubator-trafodion pull request #1113: [TRAFODION-2636] Memory leak in meta...

2017-06-07 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1113#discussion_r120685625 --- Diff: core/sql/common/NAMemory.h --- @@ -468,6 +468,11 @@ NA_EIDPROC NAMemoryType getType() { return type_

[GitHub] incubator-trafodion pull request #1113: [TRAFODION-2636] Memory leak in meta...

2017-06-07 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1113#discussion_r120684023 --- Diff: core/sql/common/NAMemory.h --- @@ -468,6 +468,11 @@ NA_EIDPROC NAMemoryType getType() { return type_

[GitHub] incubator-trafodion pull request #1113: [TRAFODION-2636] Memory leak in meta...

2017-06-07 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1113#discussion_r120683604 --- Diff: core/sql/optimizer/ControlDB.cpp --- @@ -108,7 +108,12 @@ ControlDB::~ControlDB() void ControlDB::setRequiredShape

[GitHub] incubator-trafodion pull request #1108: [TRAFODION-2631] Disk IO counter is ...

2017-06-02 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1108 [TRAFODION-2631] Disk IO counter is not populated for hdfs/hive IOs You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafodion pull request #1098: TRAFODION-2385 jdbcT4 support boolea...

2017-05-23 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1098#discussion_r118051474 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4PreparedStatement.java --- @@ -1051,10 +1054,29 @@ public void

[GitHub] incubator-trafodion pull request #1087: [TRAFODION-2420] RMS enhancements

2017-05-09 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1087#discussion_r115612077 --- Diff: core/sql/executor/ex_hash_grby.h --- @@ -302,6 +312,7 @@ NA_EIDPROC void resetClusterAndReadFromChild(); // Tmobile

[GitHub] incubator-trafodion pull request #1087: [TRAFODION-2420] RMS enhancements

2017-05-09 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1087#discussion_r115611428 --- Diff: core/sql/executor/ex_hash_grby.cpp --- @@ -452,6 +463,8 @@ short ex_hash_grby_tcb::work() { hasFreeTupp_

[GitHub] incubator-trafodion pull request #1087: [TRAFODION-2420] RMS enhancements

2017-05-09 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1087#discussion_r115610081 --- Diff: core/sql/executor/ex_hash_grby.cpp --- @@ -452,6 +463,8 @@ short ex_hash_grby_tcb::work() { hasFreeTupp_

[GitHub] incubator-trafodion pull request #1087: [TRAFODION-2420] RMS enhancements

2017-05-08 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1087 [TRAFODION-2420] RMS enhancements New counters introduced as part of BMO InterimRowCount - Counter to keep track intermediate rows produced by hash

[GitHub] incubator-trafodion pull request #1083: [TRAFODION-2098] Move all hdfs direc...

2017-05-05 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1083#discussion_r115024297 --- Diff: core/sql/sqlcomp/nadefaults.cpp --- @@ -3378,7 +3378,7 @@ XDDkwd__(SUBQUERY_UNNESTING, "ON"),

[GitHub] incubator-trafodion pull request #1083: [TRAFODION-2098] Move all hdfs direc...

2017-05-05 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1083#discussion_r115023412 --- Diff: core/sql/sqlcomp/nadefaults.cpp --- @@ -1956,7 +1956,7 @@ SDDkwd__(EXE_DIAGNOSTIC_EVENTS, "OFF"),

[GitHub] incubator-trafodion pull request #1070: [TRAFODION-2596] Improve the log4j a...

2017-04-19 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1070 [TRAFODION-2596] Improve the log4j and log4cxx infrastructure in Traf… …odion The following changes are made in the way Trafodion logs the messages: Writes

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-19 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1068#discussion_r112299127 --- Diff: core/sqf/conf/log4cxx.monitor.mon.config --- @@ -22,17 +22,16 @@ # # Define some default values that can

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-19 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1068#discussion_r112238169 --- Diff: core/sqf/conf/log4cxx.trafodion.masterexe.config --- @@ -43,19 +38,10 @@ log4j.appender.mxoAppender.addPid=false

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-19 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1068#discussion_r112237344 --- Diff: core/sqf/conf/log4cxx.monitor.mon.config --- @@ -22,17 +22,16 @@ # # Define some default values that can

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1068#discussion_r112101626 --- Diff: core/sqf/conf/log4cxx.trafodion.masterexe.config --- @@ -44,15 +45,15 @@ log4j.appender.mxoAppender.layout

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-18 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1068#discussion_r112101144 --- Diff: core/sqf/conf/log4cxx.trafodion.masterexe.config --- @@ -44,15 +45,15 @@ log4j.appender.mxoAppender.layout

[GitHub] incubator-trafodion pull request #1068: [TRAFODION-2594] Trafodion logs the ...

2017-04-17 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1068 [TRAFODION-2594] Trafodion logs the same message twice in most of its… … log4j and log4cxx logs [TRAFODION-2420] RMS enhancements [TRAFODION-2478] Memory monitor

[GitHub] incubator-trafodion pull request #1055: [TRAFODION-2483] Trafodion treat '\'...

2017-04-11 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1055#discussion_r110909382 --- Diff: core/sql/executor/ExHdfsScan.cpp --- @@ -1653,7 +1653,7 @@ char * ExHdfsScanTcb::extractAndTransformAsciiSourceToSqlRow(int

[GitHub] incubator-trafodion pull request #1052: [TRAFODION-2420] RMS enhancements

2017-04-07 Thread selvaganesang
GitHub user selvaganesang opened a pull request: https://github.com/apache/incubator-trafodion/pull/1052 [TRAFODION-2420] RMS enhancements Improved BMO operation accounting. Two new counters are added a) scratchIOSize - Size in KB of the scratch IO b) scratchIOMaxTime - Time

  1   2   3   4   >