[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654665 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLtable.cpp --- @@ -12940,6 +12940,142 @@ TrafDesc *CmpSeabaseDDL::getSeabaseRoutineDescInternal

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654612 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLtable.cpp --- @@ -12940,6 +12940,142 @@ TrafDesc *CmpSeabaseDDL::getSeabaseRoutineDescInternal

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654557 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLtable.cpp --- @@ -12940,6 +12940,142 @@ TrafDesc *CmpSeabaseDDL::getSeabaseRoutineDescInternal

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654517 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLschema.cpp --- @@ -445,13 +445,36 @@ Int16 status = ComUser::getAuthNameFromAuthID(objectOwner

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654478 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLmd.h --- @@ -115,7 +115,8 @@ static const QString seabaseColumnsDDL

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654412 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommentOn.cpp --- @@ -0,0 +1,202

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654381 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommentOn.cpp --- @@ -0,0 +1,202

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654441 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommentOn.cpp --- @@ -0,0 +1,202

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654287 --- Diff: core/sql/sqlcomp/CmpDescribe.cpp --- @@ -4129,6 +4219,33 @@ char buf[1000]; ((CmpCommon::getDefault

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654201 --- Diff: core/sql/sqlcomp/CmpDescribe.cpp --- @@ -3777,6 +3808,65 @@ short CmpDescribeSeabaseTable

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654147 --- Diff: core/sql/sqlcomp/CmpDescribe.cpp --- @@ -3059,6 +3061,36 @@ short CmpDescribeSeabaseTable ( outputLongLine

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654170 --- Diff: core/sql/sqlcomp/CmpDescribe.cpp --- @@ -3059,6 +3061,36 @@ short CmpDescribeSeabaseTable ( outputLongLine

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654096 --- Diff: core/sql/parser/sqlparser.y --- @@ -33167,7 +33173,102 @@ stats_merge_clause : /*empty

[GitHub] incubator-trafodion pull request #1288: New COMMENT-ON SQL statement

2017-11-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1288#discussion_r148654006 --- Diff: core/sql/parser/BindStmtDDL.cpp --- @@ -2576,6 +2576,60 @@ StmtDDLAlterUser::bindNode(BindWA * pBindWA

[GitHub] incubator-trafodion pull request #1282: Trafodion-2705 user has "SHOW" privi...

2017-10-29 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1282 Trafodion-2705 user has "SHOW" privilege can't do 'showddl user' Showddl code now checks to see if current user matches the user name specified in the showd

[GitHub] incubator-trafodion pull request #1274: Miscellaneous authorization changes:

2017-10-23 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1274 Miscellaneous authorization changes: - Unregister user does not remove component privileges - Reuse unused entries from the authID ranges - Add "changeuser&quo

[GitHub] incubator-trafodion pull request #1153: Fix TEST125 expected results

2017-06-28 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1153 Fix TEST125 expected results You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator-trafodion fixes

[GitHub] incubator-trafodion pull request #1151: Update expected results file for pri...

2017-06-27 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1151 Update expected results file for priv1/TEST125 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator

[GitHub] incubator-trafodion pull request #1109: TRAFODION - 2632 Performing update s...

2017-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1109#discussion_r120394504 --- Diff: core/sql/ustat/hs_globals.cpp --- @@ -3776,7 +3776,7 @@ NABoolean HSGlobalsClass::isAuthorized(NABoolean isShowStats

[GitHub] incubator-trafodion pull request #1109: TRAFODION - 2632 Performing update s...

2017-06-02 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1109 TRAFODION - 2632 Performing update statistics on metadata tables caus… …es issues Added code to return error and cleanup code a bit: *** ERROR[9205] UPDATE

[GitHub] incubator-trafodion pull request #1080: [TRAFODION-2603] Remove obsolete uti...

2017-05-02 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1080#discussion_r114438388 --- Diff: core/sql/parser/ParKeyWords.cpp --- @@ -832,7 +832,6 @@ ParKeyWord ParKeyWords::keyWords_[] = { ParKeyWord("PROTE

[GitHub] incubator-trafodion pull request #1065: Fix privs regression test scripts

2017-04-14 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1065 Fix privs regression test scripts You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator-trafodion

[GitHub] incubator-trafodion pull request #1010: TRAFODION-2538 Revoking privileges f...

2017-03-14 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/1010 TRAFODION-2538 Revoking privileges from role not invoking query inval… …idation Fixed a issue where query invalidation keys were not being sent correctly when

[GitHub] incubator-trafodion pull request #997: Fix regression failures for privs1 an...

2017-03-07 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/997 Fix regression failures for privs1 and privs2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator

[GitHub] incubator-trafodion pull request #996: Fix regression issue with privs1 and ...

2017-03-07 Thread robertamarton
Github user robertamarton closed the pull request at: https://github.com/apache/incubator-trafodion/pull/996 --- 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 #996: Fix regression issue with privs1 and ...

2017-03-07 Thread robertamarton
GitHub user robertamarton reopened a pull request: https://github.com/apache/incubator-trafodion/pull/996 Fix regression issue with privs1 and privs2 Always call reg_users.sql before running privilege tests. It does not hurt to run this scripts even if authorization already

[GitHub] incubator-trafodion pull request #996: Fix regression issue with privs1 and ...

2017-03-07 Thread robertamarton
Github user robertamarton closed the pull request at: https://github.com/apache/incubator-trafodion/pull/996 --- 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 #996: Fix regression issue with privs1 and ...

2017-03-07 Thread robertamarton
GitHub user robertamarton reopened a pull request: https://github.com/apache/incubator-trafodion/pull/996 Fix regression issue with privs1 and privs2 Always call reg_users.sql before running privilege tests. It does not hurt to run this scripts even if authorization already

[GitHub] incubator-trafodion pull request #996: Fix regression issue with privs1 and ...

2017-03-07 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/996 Fix regression issue with privs1 and privs2 Always call reg_users.sql before running privilege tests. It does not hurt to run this scripts even if authorization already

[GitHub] incubator-trafodion pull request #979: TRAFODION-2498 Add support to run hiv...

2017-02-27 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/979#discussion_r103294768 --- Diff: core/sql/executor/ExExeUtilMisc.cpp --- @@ -2509,3 +2509,128 @@ ExExeUtilHiveTruncatePrivateState

[GitHub] incubator-trafodion pull request #975: TRAFODION-2441 user has only select p...

2017-02-22 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/975 TRAFODION-2441 user has only select privilege on a table can do ... This is the second delivery that updates "get" commands so users can only view information where

[GitHub] incubator-trafodion pull request #966: Added FILTER144 to privs2 regression

2017-02-16 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/966 Added FILTER144 to privs2 regression You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator-trafodion

[GitHub] incubator-trafodion pull request #957: TRAFODION-2441 user has only select p...

2017-02-15 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/957#discussion_r101307451 --- Diff: core/sql/common/ComUser.cpp --- @@ -348,6 +348,33 @@ Int16 ComUser::getAuthNameFromAuthID(Int32 authID, return FEOK

[GitHub] incubator-trafodion pull request #957: TRAFODION-2441 user has only select p...

2017-02-13 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/957 TRAFODION-2441 user has only select privilege on a table can do ... TRAFODION-2409 support privilege control(column privileges) for hive tables TRAFODION-2423 any user can

[GitHub] incubator-trafodion pull request #937: Trafodion-2175

2017-01-30 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/937 Trafodion-2175 Added new regression test privs1/TEST123 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton

[GitHub] incubator-trafodion pull request #932: TRAFODION-2175 a user should only see...

2017-01-26 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/932 TRAFODION-2175 a user should only see specific schemas/tables that he… … has privs TRAFODION-1573 Additional GET commands for privileges This is one

[GitHub] incubator-trafodion pull request #870: [TRAFODION-2389] Update some messages...

2016-12-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/870#discussion_r91203013 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -254,6 +254,24 @@ local super ID can perform a replication

[GitHub] incubator-trafodion pull request #870: [TRAFODION-2389] Update some messages...

2016-12-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/870#discussion_r91204885 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -486,62 +534,52 @@ catalog name and resubmit. == SQL 1036

[GitHub] incubator-trafodion pull request #870: [TRAFODION-2389] Update some messages...

2016-12-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/870#discussion_r91205088 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -486,62 +534,52 @@ catalog name and resubmit. == SQL 1036

[GitHub] incubator-trafodion pull request #870: [TRAFODION-2389] Update some messages...

2016-12-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/870#discussion_r91205877 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -761,40 +814,42 @@ constraint specified in the CREATE INDEX

[GitHub] incubator-trafodion pull request #855: Traf 2357

2016-11-28 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/855 Traf 2357 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator-trafodion traf-2357 Alternatively you can

[GitHub] incubator-trafodion pull request #845: TRAFODION-2156 Update SQL reference m...

2016-11-21 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/845 TRAFODION-2156 Update SQL reference manual for column level privileges You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafodion pull request #806: TRAFODION-2330 Using trafci, a select...

2016-10-31 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/806 TRAFODION-2330 Using trafci, a select from a table succeeds even if t… …he user does not have the priv There is a problem when the session

[GitHub] incubator-trafodion pull request #803: TRAFODION-2327 Reduce I/O when loadin...

2016-10-31 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/803 TRAFODION-2327 Reduce I/O when loading objects into caches For each authorization ID (user, role, or PUBLIC), a bitmap containing the accumulated privileges (across all

[GitHub] incubator-trafodion pull request #781: Added regression test privs2/TEST144

2016-10-25 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/781 Added regression test privs2/TEST144 Added regression test privs2/TEST144 and its helpers that contains grant and revoke tests for functions and procedures. You can merge

[GitHub] incubator-trafodion pull request #766: [TRAFODION-2282][TRAFODION-882] New s...

2016-10-19 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/766#discussion_r84122119 --- Diff: core/sql/optimizer/NATable.cpp --- @@ -275,75 +282,28 @@ void HistogramCache::getHistograms(NATable& table) // loo

[GitHub] incubator-trafodion pull request #765: [TRAFODION-1758]: A user has dbroot r...

2016-10-17 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/765 [TRAFODION-1758]: A user has dbroot role can't grant component privilege Privilege checks were not handling role checks correctly. PrivMgrComponentPrivileges::hasWGO is now

[GitHub] incubator-trafodion pull request #761: TRAFODION-2281 - Unable to install Ke...

2016-10-12 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/761 TRAFODION-2281 - Unable to install Kerberos Fixed a couple of problems with security installation scripts: - Need to source config file in traf_secure_config before

[GitHub] incubator-trafodion pull request #760: [TRAFODION-2267] Fix ALTER TABLE RENA...

2016-10-12 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/760#discussion_r83070635 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -3044,3 +3044,146 @@ statement. *Recovery:* Remove

[GitHub] incubator-trafodion pull request #760: [TRAFODION-2267] Fix ALTER TABLE RENA...

2016-10-12 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/760#discussion_r83070053 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -3044,3 +3044,146 @@ statement. *Recovery:* Remove

[GitHub] incubator-trafodion pull request #760: [TRAFODION-2267] Fix ALTER TABLE RENA...

2016-10-12 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/760#discussion_r83069409 --- Diff: docs/messages_guide/src/asciidoc/_chapters/ddl_msgs.adoc --- @@ -3044,3 +3044,146 @@ statement. *Recovery:* Remove

[GitHub] incubator-trafodion pull request #757: [TRAFODION-2167]: Invalid query inval...

2016-10-11 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/757 [TRAFODION-2167]: Invalid query invalidation keys not working properly When a user is revoked from a role, invalidation keys are not being processed correctly. Therefore

[GitHub] incubator-trafodion pull request #729: TRAFODION-2203 - a user can grant pri...

2016-09-28 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/729#discussion_r80934472 --- Diff: core/sql/sqlcomp/PrivMgrPrivileges.cpp --- @@ -3182,34 +3042,107 @@ PrivStatus PrivMgrPrivileges::revokeColumnPrivileges

[GitHub] incubator-trafodion pull request #729: TRAFODION-2203 - a user can grant pri...

2016-09-26 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/729 TRAFODION-2203 - a user can grant privileges that he doesn’t have ... ... to other users/roles successfully In this case, the user/role did not get the privilege requested even

[GitHub] incubator-trafodion pull request #704: [TRAFODION-1794]: Log authentication ...

2016-09-20 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/704#discussion_r79636749 --- Diff: core/dbsecurity/auth/inc/authEvents.h --- @@ -0,0 +1,89

[GitHub] incubator-trafodion pull request #704: [TRAFODION-1794]: Log authentication ...

2016-09-20 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/704#discussion_r79636671 --- Diff: core/dbsecurity/auth/Makefile --- @@ -70,10 +67,11 @@ INCLUDES= -I. -I./inc -I ../shared/inc \ -I

[GitHub] incubator-trafodion pull request #704: [TRAFODION-1794]: Log authentication ...

2016-09-14 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/704 [TRAFODION-1794]: Log authentication Information Added code to log LDAP connection errors into the associated log4cxx file. For authentications that occur in mxosrvr, events

[GitHub] incubator-trafodion pull request #660: TRAFODION [2137] Improve metadata acc...

2016-08-18 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/660#discussion_r75333183 --- Diff: core/sql/comexe/ComTdb.h --- @@ -1143,6 +1145,33 @@ class ComTdbVirtTableSequenceInfo : public ComTdbVirtTableBase Int64

[GitHub] incubator-trafodion pull request #660: TRAFODION [2137] Improve metadata acc...

2016-08-18 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/660#discussion_r75329401 --- Diff: core/sql/generator/Generator.cpp --- @@ -1737,6 +1737,82 @@ TrafDesc * Generator::createConstrKeyColsDescs(Int32 numKeys

[GitHub] incubator-trafodion pull request #660: TRAFODION [2137] Improve metadata acc...

2016-08-18 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/660#discussion_r75329351 --- Diff: core/sql/common/ComSecurityKey.cpp --- @@ -73,6 +74,59 @@ NABoolean qiCheckForInvalidObject (const Int32 numInvalidationKeys

[GitHub] incubator-trafodion pull request #661: [TRAFODION-2161] Support migration of...

2016-08-17 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/661#discussion_r75234600 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLrepos.cpp --- @@ -358,6 +358,75 @@ short CmpSeabaseDDL::copyOldReposToNew(ExeCliInterface

[GitHub] incubator-trafodion pull request #661: [TRAFODION-2161] Support migration of...

2016-08-17 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/661#discussion_r75234375 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLrepos.cpp --- @@ -516,6 +585,62 @@ short CmpSeabaseDDL::upgradeReposComplete(ExeCliInterface

[GitHub] incubator-trafodion pull request #661: [TRAFODION-2161] Support migration of...

2016-08-17 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/661#discussion_r75232143 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLrepos.cpp --- @@ -516,6 +585,62 @@ short CmpSeabaseDDL::upgradeReposComplete(ExeCliInterface

[GitHub] incubator-trafodion pull request #656: [TRAFODION-2160]: Metadata upgrade fa...

2016-08-12 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/656 [TRAFODION-2160]: Metadata upgrade failed with internal error in Priv… …MgrMD.cpp Fixed a check that verifies component privilege grants were sufficient during

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

2016-08-05 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/644 Trafodion 1788 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton/incubator-trafodion trafodion-1788

[GitHub] incubator-trafodion pull request #565: [TRAFODION-2037] Improve DDL concurre...

2016-07-14 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/565#discussion_r70833655 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLrepos.cpp --- @@ -114,43 +123,72 @@ short CmpSeabaseDDL::createRepos(ExeCliInterface

[GitHub] incubator-trafodion pull request #565: [TRAFODION-2037] Improve DDL concurre...

2016-07-14 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/565#discussion_r70833180 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLrepos.cpp --- @@ -47,8 +47,17 @@ short CmpSeabaseDDL::createRepos(ExeCliInterface * cliInterface

[GitHub] incubator-trafodion pull request #565: [TRAFODION-2037] Improve DDL concurre...

2016-07-14 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/565#discussion_r70833027 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLmd.h --- @@ -661,6 +690,452 @@ static const MDTableInfo allMDHistInfo[] = { #define

[GitHub] incubator-trafodion pull request #565: [TRAFODION-2037] Improve DDL concurre...

2016-07-14 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/565#discussion_r70831255 --- Diff: core/sql/parser/sqlparser.y --- @@ -16026,13 +16027,30 @@ exe_util_init_hbase : TOK_INITIALIZE TOK_TRAFODION

[GitHub] incubator-trafodion pull request #578: Updated release notes and download pa...

2016-07-06 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/578 Updated release notes and download page for release 2.0.1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/robertamarton

[GitHub] incubator-trafodion pull request #558: TRAFODION [2056] Update installation ...

2016-06-28 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/558#discussion_r68763759 --- Diff: docs/provisioning_guide/src/asciidoc/_chapters/enable_security.adoc --- @@ -26,25 +26,100 @@ [[enable-security

[GitHub] incubator-trafodion pull request #557: Fixes for TRAFODION-2068

2016-06-22 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/557 Fixes for TRAFODION-2068 TRAFODION-2068: Missing DISCLAIMER files for release package Added DISCLAIMER file to servers package. You can merge this pull request

[GitHub] incubator-trafodion pull request #546: TRAFODION [109] Instrument Trafodion ...

2016-06-22 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/546#discussion_r68089548 --- Diff: install/installer/traf_secure_setup --- @@ -0,0 +1,342 @@ +#!/bin/bash + +# @@@ START COPYRIGHT

[GitHub] incubator-trafodion pull request #554: Fixes for TRAFODION-2068 and TRAFODIO...

2016-06-21 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/554 Fixes for TRAFODION-2068 and TRAFODION-2082 TRAFODION-2068: Missing DISCLAIMER files for release package TRAFODION-2082: BSD-4 license for swsprintf and swscanf unclear

[GitHub] incubator-trafodion pull request #553: Fixes for TRAFODION-2068 and TRAFODIO...

2016-06-21 Thread robertamarton
Github user robertamarton closed the pull request at: https://github.com/apache/incubator-trafodion/pull/553 --- 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 #553: Fixes for TRAFODION-2068 and TRAFODIO...

2016-06-21 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/553 Fixes for TRAFODION-2068 and TRAFODION-2082 TRAFODION-2068: Missing DISCLAIMER files for release package TRAFODION-2082: BSD-4 license for swsprintf and swscanf unclear

[GitHub] incubator-trafodion pull request #546: TRAFODION [109] Instrument Trafodion ...

2016-06-15 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/546#discussion_r67192402 --- Diff: core/sqf/sql/scripts/sqstart --- @@ -220,6 +220,31 @@ function SQCheckOrphanProcesses { } +function

[GitHub] incubator-trafodion pull request #546: TRAFODION [109] Instrument Trafodion ...

2016-06-14 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/546 TRAFODION [109] Instrument Trafodion to work with Secure Hadoop (Kerberos) The first phase of automation with Kerberos is ready including: New secure install: Customer

[GitHub] incubator-trafodion pull request #544: [TRAFODION-2044] Support SALT clause ...

2016-06-13 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/544#discussion_r66873667 --- Diff: core/sql/parser/sqlparser.y --- @@ -24909,7 +24946,6 @@ create_table_as_token: TOK_AS /* type pElemDDL

[GitHub] incubator-trafodion pull request #544: [TRAFODION-2044] Support SALT clause ...

2016-06-13 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/544#discussion_r66870357 --- Diff: core/sql/regress/seabase/TEST040 --- @@ -0,0 +1,139 @@ +-- test for CREATE TABLE LIKE +-- +-- @@@ START COPYRIGHT

[GitHub] incubator-trafodion pull request #544: [TRAFODION-2044] Support SALT clause ...

2016-06-13 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/544#discussion_r66868775 --- Diff: core/sql/parser/sqlparser.y --- @@ -24909,7 +24946,6 @@ create_table_as_token: TOK_AS /* type pElemDDL

[GitHub] incubator-trafodion pull request #538: TRAFODION [109] Instrument Trafodion ...

2016-06-10 Thread robertamarton
GitHub user robertamarton opened a pull request: https://github.com/apache/incubator-trafodion/pull/538 TRAFODION [109] Instrument Trafodion to work with Secure Hadoop (Kerberos) The first phase of automation with Kerberos is ready. The following use cases are supported

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-07 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r66085087 --- Diff: core/sql/cli/Context.cpp --- @@ -3850,6 +3856,48 @@ void ContextCli::createMxcmpSession() mxcmpSessionInUse_ = TRUE

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65936988 --- Diff: core/sql/arkcmp/CmpStatement.cpp --- @@ -1254,17 +1255,34 @@ CmpStatement::process(const CmpMessageDatabaseUser

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65936862 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp --- @@ -7764,52 +7734,103 @@ short CmpSeabaseDDL::initSeabaseAuthorization

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65935714 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp --- @@ -7764,52 +7734,103 @@ short CmpSeabaseDDL::initSeabaseAuthorization

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65935336 --- Diff: core/sql/cli/Context.cpp --- @@ -3850,6 +3856,48 @@ void ContextCli::createMxcmpSession() mxcmpSessionInUse_ = TRUE

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65934536 --- Diff: core/sql/cli/Context.cpp --- @@ -3850,6 +3856,48 @@ void ContextCli::createMxcmpSession() mxcmpSessionInUse_ = TRUE

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65933098 --- Diff: core/sql/cli/Context.cpp --- @@ -3718,19 +3719,27 @@ void ContextCli::createMxcmpSession() return; // Steps

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65900595 --- Diff: core/sql/cli/Context.cpp --- @@ -3850,6 +3856,48 @@ void ContextCli::createMxcmpSession() mxcmpSessionInUse_ = TRUE

[GitHub] incubator-trafodion pull request #523: TRAFODION [2025] Initialize authoriza...

2016-06-06 Thread robertamarton
Github user robertamarton commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/523#discussion_r65899408 --- Diff: core/sql/cli/Context.cpp --- @@ -3850,6 +3856,48 @@ void ContextCli::createMxcmpSession() mxcmpSessionInUse_ = TRUE

  1   2   >