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

2016-10-18 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-trafodion/pull/765


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 checking privileges against roles.
privs1/TEST137 was updated to test role privileges
priv1/TEST120  was added to test query invalidation with roles
   (forgot to add it for a previous check in)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/robertamarton/incubator-trafodion 
trafodion-1758

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/765.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #765


commit 1c93857502acd9ef9e14154df6a6e397284acc81
Author: Roberta Marton 
Date:   2016-10-17T20:30:59Z

[TRAFODION-1758]: A user has dbroot role can't grant component privilege

Privilege checks were not handling role checks correctly.
PrivMgrComponentPrivileges::hasWGO is now checking privileges against roles.
privs1/TEST137 was updated to test role privileges
priv1/TEST120  was added to test query invalidation with roles
   (forgot to add it for a previous check in)




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---