[GitHub] incubator-trafodion pull request #648: fix jira 2512

2016-08-08 Thread AlexPeng19
GitHub user AlexPeng19 opened a pull request: https://github.com/apache/incubator-trafodion/pull/648 fix jira 2512 fix jira 2512 You can merge this pull request into a Git repository by running: $ git pull https://github.com/AlexPeng19/incubator-trafodion master Alternatively

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

2016-08-08 Thread selvaganesang
Github user selvaganesang commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/645#discussion_r73978339 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -2288,

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

2016-08-08 Thread trinakrug
Github user trinakrug commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/645#discussion_r73977162 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -2288,19 +

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

2016-08-08 Thread trinakrug
Github user trinakrug commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/645#discussion_r73977174 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -2327,21 +

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

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/647#discussion_r73971774 --- Diff: core/sql/generator/GenResources.cpp --- @@ -97,92 +77,74 @@ static ExScratchDiskDrive * genScratchDriveList(const NAString &def,

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

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/647#discussion_r73970664 --- Diff: core/sql/generator/GenResources.cpp --- @@ -97,92 +77,74 @@ static ExScratchDiskDrive * genScratchDriveList(const NAString &def,

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

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/647#discussion_r73970577 --- Diff: core/sql/generator/GenResources.cpp --- @@ -83,12 +62,13 @@ static ExScratchDiskDrive * genScratchDriveList(const NAString &def,

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

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

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

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/647#discussion_r73969461 --- Diff: core/sqf/src/stfs/stfs_stub.cpp --- @@ -115,6 +115,7 @@ namespace STFS_stub { } stfs_directories; stfs_director

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

2016-08-08 Thread sandhyasun
GitHub user sandhyasun opened a pull request: https://github.com/apache/incubator-trafodion/pull/647 [TRAFODION-2146] Use of CQD to set scratch directory locations Scratchfile creation cleanup and new code to use CQD SCRATCH_DISKS to specify scratch disk locations for overflow. Remo

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73957574 --- Diff: core/sql/sqlcomp/PrivMgrPrivileges.cpp --- @@ -4208,43 +4514,59 @@ void PrivMgrPrivileges::summarizeColPrivs( // get COLUM

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73955914 --- Diff: core/sql/sqlcomp/PrivMgrPrivileges.cpp --- @@ -2101,17 +2093,114 @@ PrivStatus PrivMgrPrivileges::dealWithViews( pUsag

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73955676 --- Diff: core/sql/sqlcomp/PrivMgrPrivileges.cpp --- @@ -2101,17 +2093,114 @@ PrivStatus PrivMgrPrivileges::dealWithViews( pUsag

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73954547 --- Diff: core/sql/sqlcomp/PrivMgrMD.h --- @@ -187,6 +202,25 @@ class ObjectUsage PrivMgrDesc originalPrivs; PrivMgrDesc update

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73954389 --- Diff: core/sql/sqlcomp/PrivMgrMD.h --- @@ -187,6 +202,25 @@ class ObjectUsage PrivMgrDesc originalPrivs; PrivMgrDesc update

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73953757 --- Diff: core/sql/sqlcomp/PrivMgrMD.cpp --- @@ -630,6 +630,138 @@ bool PrivMgrMDAdmin::isAuthorized (void) } // ***

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73953448 --- Diff: core/sql/sqlcomp/PrivMgrMD.cpp --- @@ -630,6 +630,138 @@ bool PrivMgrMDAdmin::isAuthorized (void) } // ***

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73953193 --- Diff: core/sql/sqlcomp/PrivMgrMD.cpp --- @@ -630,6 +630,138 @@ bool PrivMgrMDAdmin::isAuthorized (void) } // ***

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73952273 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLview.cpp --- @@ -394,8 +458,7 @@ short CmpSeabaseDDL::gatherViewPrivileges (const StmtDDLCreateView

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73952091 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLview.cpp --- @@ -247,6 +249,65 @@ short CmpSeabaseDDL::buildViewColInfo(StmtDDLCreateView * createVi

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73949536 --- Diff: core/sql/common/ComViewColUsage.h --- @@ -0,0 +1,89 @@ +// @@@ START COPYRIGHT @@@ +// +// Licensed to the Apache Softw

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73949412 --- Diff: core/sql/common/ComViewColUsage.h --- @@ -0,0 +1,89 @@ +// @@@ START COPYRIGHT @@@ +// +// Licensed to the Apache Softw

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

2016-08-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-trafodion/pull/645 --- 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, or if the fe

[GitHub] incubator-trafodion pull request #644: Trafodion 1788

2016-08-08 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/644#discussion_r73945048 --- Diff: core/sql/bin/SqlciErrors.txt --- @@ -26,7 +26,7 @@ 1024 Z 9 BEGINNER MINOR DBADMIN File system error $0~NSKCode occur

[GitHub] incubator-trafodion pull request #646: TRAFODION-2148 fix

2016-08-08 Thread prashanth-vasudev
GitHub user prashanth-vasudev opened a pull request: https://github.com/apache/incubator-trafodion/pull/646 TRAFODION-2148 fix SQL calls create and drop requests to transaction manager. However, sometimes in error scenarios, the error codes never reach SQL. This results in sql assu

[GitHub] incubator-trafodion pull request #642: [TRAFODION-2119] create table report ...

2016-08-08 Thread sureshsubbiah
Github user sureshsubbiah commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/642#discussion_r73875322 --- Diff: core/sql/common/CharType.cpp --- @@ -1217,7 +1217,7 @@ void CharType::minMaxRepresentableValue(void* bufPtr, if (stringLite

[GitHub] incubator-trafodion pull request #639: [TRAFODION-2117] add SQL support of E...

2016-08-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-trafodion/pull/639 --- 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, or if the fe

[GitHub] incubator-trafodion pull request #643: [TRAFODION-2145]sqlcolumns returned e...

2016-08-08 Thread xwq
Github user xwq commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/643#discussion_r73836940 --- Diff: core/conn/odbc/src/odbc/nsksrvrcore/srvrothers.cpp --- @@ -4834,20 +4834,19 @@ odbc_SQLSvc_GetSQLCatalogs_sme_( "('DECIMAL', 3, 18, N

[GitHub] incubator-trafodion pull request #643: [TRAFODION-2145]sqlcolumns returned e...

2016-08-08 Thread xwq
Github user xwq commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/643#discussion_r73836469 --- Diff: core/conn/odbc/src/odbc/nsksrvrcore/srvrothers.cpp --- @@ -4834,20 +4834,19 @@ odbc_SQLSvc_GetSQLCatalogs_sme_( "('DECIMAL', 3, 18, N