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

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

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

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

[GitHub] incubator-trafodion pull request #1093: [Trafodion-2608] Skips file flush in...

2017-05-11 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/1093 [Trafodion-2608] Skips file flush in case of insert drop same transaction You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafodion pull request #1075: This is a large contribution of chan...

2017-05-08 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1075#discussion_r115299127 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #1075: This is a large contribution of chan...

2017-05-08 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1075#discussion_r115295938 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TmDDL.java --- @@ -80,11

[GitHub] incubator-trafodion pull request #1081: [TRAFODION-2604] sort merge phase me...

2017-05-05 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1081#discussion_r115067125 --- Diff: core/sql/executor/ex_sort.cpp --- @@ -218,7 +304,8 @@ ExSortTcb::ExSortTcb(const ExSortTdb & sort_tdb, sortSt

[GitHub] incubator-trafodion pull request #1081: [TRAFODION-2604] sort merge phase me...

2017-05-02 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/1081 [TRAFODION-2604] sort merge phase memory pool improvements You can merge this pull request into a Git repository by running: $ git pull https://github.com/prashanth

[GitHub] incubator-trafodion pull request #1006: [TRAFODION-2523] Fixes drop schema c...

2017-03-13 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/1006 [TRAFODION-2523] Fixes drop schema cascade memory corruption. You can merge this pull request into a Git repository by running: $ git pull https://github.com/prashanth

[GitHub] incubator-trafodion pull request #993: [TRAFODION-2468] DDL HA , step 2 chan...

2017-03-06 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/993 [TRAFODION-2468] DDL HA , step 2 changes(please refer jira). You can merge this pull request into a Git repository by running: $ git pull https://github.com/prashanth

[GitHub] incubator-trafodion pull request #953: [TRAFODION-2478] Reduce the number of...

2017-02-13 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/953#discussion_r100663267 --- Diff: core/sql/bin/ex_esp_main.cpp --- @@ -357,15 +357,18 @@ Int32 runESP(Int32 argc, char** argv, GuaReceiveFastStart

[GitHub] incubator-trafodion pull request #947: [TRAFODION-2472] disable transactiona...

2017-02-07 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/947 [TRAFODION-2472] disable transactional alter command for hbase options. Transaction DDL for alter commands is currently disabled. There are few statements such as alter

[GitHub] incubator-trafodion pull request #909: [TRAFODION-2433] Capture and log call...

2017-02-01 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/909#discussion_r98937516 --- Diff: core/sql/arkcmp/CompException.cpp --- @@ -113,11 +114,21 @@ FatalException::FatalException(const char *msg, else

[GitHub] incubator-trafodion pull request #936: [TRAFODION-2466]Handle sortReclen lim...

2017-01-31 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/936#discussion_r98654201 --- Diff: core/sql/generator/GenRelMisc.cpp --- @@ -3071,7 +3074,16 @@ short Sort::generateTdb(Generator * generator

[GitHub] incubator-trafodion pull request #936: [TRAFODION-2466]Handle sortReclen lim...

2017-01-30 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/936#discussion_r98557525 --- Diff: core/sql/sqlcomp/DefaultConstants.h --- @@ -2585,6 +2585,7 @@ enum DefaultConstants SCRATCH_MGMT_OPTION

[GitHub] incubator-trafodion pull request #936: [TRAFODION-2466]Handle sortReclen lim...

2017-01-30 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/936#discussion_r98557444 --- Diff: core/sql/generator/GenRelMisc.cpp --- @@ -3071,7 +3074,16 @@ short Sort::generateTdb(Generator * generator

[GitHub] incubator-trafodion pull request #909: [TRAFODION-2433] Capture and log call...

2017-01-30 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/909#discussion_r98519641 --- Diff: core/sql/arkcmp/CompException.cpp --- @@ -113,11 +114,21 @@ FatalException::FatalException(const char *msg, else

[GitHub] incubator-trafodion pull request #936: [TRAFODION-2466]Handle sortReclen lim...

2017-01-28 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/936 [TRAFODION-2466]Handle sortReclen limit in sort overflow. Handle sortReclen limit in sort overflow. Also minor change to commit conflict error text. You can merge

[GitHub] incubator-trafodion pull request #917: [TRAFODION-2150] and [TRAFODION-2151]...

2017-01-18 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/917 [TRAFODION-2150] and [TRAFODION-2151] TM long prepare and chore thread Issues Queries such as drop schema cascase involve multiple DDL operations that take some time

[GitHub] incubator-trafodion pull request #907: [TRAFODION-2259] Add TopN stats count...

2017-01-09 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/907 [TRAFODION-2259] Add TopN stats counter and realign BMO stats. As part of TopN sort feature , adding a stats entry to represent Topn. Also realigned the bmo stats columns

[GitHub] incubator-trafodion pull request #885: [TRAFODION-2414] enhance 8616 commit ...

2016-12-21 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/885 [TRAFODION-2414] enhance 8616 commit conflict error message to include tx conflict details commit conflict error detail is available in the region. This error is raised

[GitHub] incubator-trafodion pull request #861: [TRAFODION-1988] Better java exceptio...

2016-12-07 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/861#discussion_r91393909 --- Diff: core/sqf/src/seatrans/tm/hbasetmlib2/javaobjectinterfacetm.cpp --- @@ -352,12 +349,11 @@ JOI_RetCode JavaObjectInterfaceTM

[GitHub] incubator-trafodion pull request #860: [TRAFODION 2381] Enhance awk script t...

2016-11-30 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/860 [TRAFODION 2381] Enhance awk script to support many variations of distros Make awk script extensible to support any number of distro variations. Also support #elseif

[GitHub] incubator-trafodion pull request #725: TRAFODION-2231 support transactional ...

2016-11-28 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/725#discussion_r89842560 --- Diff: core/sqf/src/tm/tmddlrequests.cpp --- @@ -34,57 +34,68 @@ using namespace std; * Signature: ([B)V

[GitHub] incubator-trafodion pull request #725: TRAFODION-2231 support transactional ...

2016-11-22 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/725#discussion_r89240739 --- Diff: core/sqf/export/include/dtm/tm.h --- @@ -274,11 +274,19 @@ typedef struct TM_HBASEREGIONINFO { // Trafodion start

[GitHub] incubator-trafodion pull request #725: TRAFODION-2231 support transactional ...

2016-11-22 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/725#discussion_r89240857 --- Diff: core/sqf/src/seatrans/tm/hbasetmlib2/hbasetm.cpp --- @@ -913,6 +921,10 @@ int CHbaseTM::alterTable(int64 pv_transid

[GitHub] incubator-trafodion pull request #725: TRAFODION-2231 support transactional ...

2016-11-22 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/725#discussion_r89240955 --- Diff: core/sqf/src/tm/tmddlrequests.cpp --- @@ -93,108 +104,167 @@ JNIEXPORT jint JNICALL

[GitHub] incubator-trafodion pull request #725: TRAFODION-2231 support transactional ...

2016-11-22 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/725#discussion_r89240530 --- Diff: core/sqf/src/tm/tmddlrequests.cpp --- @@ -34,57 +34,68 @@ using namespace std; * Signature: ([B)V

[GitHub] incubator-trafodion pull request #828: [TRAFODION-2342] hdfs access threads ...

2016-11-14 Thread prashanth-vasudev
Github user prashanth-vasudev closed the pull request at: https://github.com/apache/incubator-trafodion/pull/828 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafodion pull request #828: [TRAFODION-2342] hdfs access threads ...

2016-11-09 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/828 [TRAFODION-2342] hdfs access threads init/cleanup changes In the effort to identify zombie threads , there were few places in code that require init and cleanup calls be made

[GitHub] incubator-trafodion pull request #815: [TRAFODION-2326] TopN sort reuse of r...

2016-11-04 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/815#discussion_r86498129 --- Diff: core/sql/executor/ex_sort.h --- @@ -356,4 +363,119 @@ class ExSortFromTopPrivateState : public ex_tcb_private_state

[GitHub] incubator-trafodion pull request #815: [TRAFODION-2326] TopN sort reuse of r...

2016-11-04 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/815#discussion_r86497624 --- Diff: core/sql/executor/ex_sort.cpp --- @@ -183,6 +212,11 @@ ExSortTcb::ExSortTcb(const ExSortTdb & sort

[GitHub] incubator-trafodion pull request #815: [TRAFODION-2326] TopN sort reuse of r...

2016-11-04 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/815#discussion_r86497128 --- Diff: core/sql/executor/ex_sort.cpp --- @@ -96,6 +96,72 @@ ExOperStats * ExSortTcb::doAllocateStatsEntry(CollHeap *heap

[GitHub] incubator-trafodion pull request #815: [TRAFODION-2326] TopN sort reuse of r...

2016-11-03 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/815 [TRAFODION-2326] TopN sort reuse of released tuples You can merge this pull request into a Git repository by running: $ git pull https://github.com/prashanth-vasudev

[GitHub] incubator-trafodion pull request #797: [TRAFODION-2326] TopN sort reuse of r...

2016-11-03 Thread prashanth-vasudev
Github user prashanth-vasudev closed the pull request at: https://github.com/apache/incubator-trafodion/pull/797 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafodion pull request #797: [TRAFODION-2326] TopN sort reuse of r...

2016-10-28 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/797 [TRAFODION-2326] TopN sort reuse of released tuples. You can merge this pull request into a Git repository by running: $ git pull https://github.com/prashanth-vasudev

[GitHub] incubator-trafodion pull request #779: [TRAFODION-2304] handle zero records ...

2016-10-23 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/779 [TRAFODION-2304] handle zero records in TOPN sort + regress TEST107 update/cleanup You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafodion pull request #768: [TRAFODION-2296] Consistent error rep...

2016-10-19 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/768#discussion_r84170605 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #768: [TRAFODION-2296] Consistent error rep...

2016-10-19 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/768#discussion_r84179202 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #768: [TRAFODION-2296] Consistent error rep...

2016-10-19 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/768#discussion_r84177699 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #768: [TRAFODION-2296] Consistent error rep...

2016-10-19 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/768#discussion_r84178660 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #768: [TRAFODION-2296] Consistent error rep...

2016-10-19 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/768#discussion_r84179508 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #:

2016-10-11 Thread prashanth-vasudev
Github user prashanth-vasudev commented on the pull request: https://github.com/apache/incubator-trafodion/commit/3cd201a6eb98ec412e33af44f106b69b215e2dcf#commitcomment-19381216 In core/sql/sort/Topn.cpp: In core/sql/sort/Topn.cpp on line 121: This has been fixed in the next

[GitHub] incubator-trafodion pull request #743: [TRAFODION-2259] TopN sort changes.

2016-10-03 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/743 [TRAFODION-2259] TopN sort changes. This includes first of changes related to sort implementation in executor. cqd gen_sort_topn_size 'N' forces sort to use topn

[GitHub] incubator-trafodion pull request #672: [TRAFODION-2170] create table failure...

2016-08-25 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/672#discussion_r76280881 --- Diff: core/sqf/src/tm/tmddlrequests.cpp --- @@ -39,13 +40,24 @@ JNIEXPORT void JNICALL

[GitHub] incubator-trafodion pull request #647: [TRAFODION-2146] Use of CQD to set sc...

2016-08-09 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/647#discussion_r74074060 --- Diff: core/sqf/src/stfs/stfs_stub.cpp --- @@ -328,63 +332,69 @@ copySTFSLocation(int pv_instnum) { //Directory

[GitHub] incubator-trafodion pull request #627: [TRAFODION-2129] Trafodion to avoid u...

2016-08-02 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/627#discussion_r73218875 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java

[GitHub] incubator-trafodion pull request #610: TRAFODION-2067 support cdh5.7 & hbase...

2016-08-02 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73198817 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/regionserver/transactional/TrxTransactionState.java.tmpl

[GitHub] incubator-trafodion pull request #610: TRAFODION-2067 support cdh5.7 & hbase...

2016-08-02 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73198489 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/regionserver/transactional/TrxTransactionState.java.tmpl

[GitHub] incubator-trafodion pull request #610: TRAFODION-2067 support cdh5.7 & hbase...

2016-08-02 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73192982 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/SsccRegionEndpoint.java.tmpl

[GitHub] incubator-trafodion pull request #610: TRAFODION-2067 support cdh5.7 & hbase...

2016-08-02 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73192266 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/TrxRegionEndpoint.java.tmpl

[GitHub] incubator-trafodion pull request #568: TRAFODION-2016

2016-07-15 Thread prashanth-vasudev
Github user prashanth-vasudev commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/568#discussion_r71025497 --- Diff: core/sqf/src/seatrans/hbase-trx/Makefile --- @@ -54,35 +65,49 @@ jdk_1_7_mapr: build_chk_mapr jdk_1_7_hdp: build_chk_hdp

[GitHub] incubator-trafodion pull request: Fix for TRAFODION-1948

2016-04-28 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/456 Fix for TRAFODION-1948 Drop schema cascade internally selects against objects table. The fix is to use a distinct clause in the internal select query. You can merge

[GitHub] incubator-trafodion pull request: TRAFODION-1798 This change is it...

2016-02-10 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/317 TRAFODION-1798 This change is item 2( of 4 items ) listed in jira Fixes the following scenario which resulted in a hang. Begin work; insert into table t values( 1

[GitHub] incubator-trafodion pull request: TRAFODION-1798- Enable DDL trans...

2016-02-05 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/309 TRAFODION-1798- Enable DDL transaction infrastructure by default. This is Step 0 ( of 4 steps ) as outlined in the jira TRAFODION-1798 You can merge this pull request

[GitHub] incubator-trafodion pull request: [TRAFODION-1490] Fix backup/rest...

2015-09-15 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/82 [TRAFODION-1490] Fix backup/restore scripts to detect CDH parcel install. You can merge this pull request into a Git repository by running: $ git pull https://github.com