trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robertamarton <...@git.apache.org>
Subject [GitHub] trafodion pull request #1717: Fixes for TRAFODION-3194 && TRAFODION-3195
Date Fri, 21 Sep 2018 22:33:16 GMT
GitHub user robertamarton opened a pull request:

    https://github.com/apache/trafodion/pull/1717

    Fixes for TRAFODION-3194 && TRAFODION-3195

    TRAFODION-3194 Revoke grant option on objects revokes more that grant option
      changed Privilege Manager to set bitmaps correctly
      removed unused methods from PrivMgrDesc
    
    TRAFODION-3195: Fixes for get commands:
    
    get schemas for user <user>:
      returns schemas owned by the specified user
      if current user does not have elevated privilege,
       returns error if current user does not match <user>.
    
    get schemas for role <role>:
      returns schemas owned by the role,
      if current user does not have elevated privilege,
        returns error if current user has not been granted <role>
    
    get [tables | views | indexes | libraries ] for user <user>:
    get [functions | table_mapping_functions | procedures] for user <user>:
    get [privileges | roles] for user <user>:
      returns objects where <user> has at least one privilege
      if current user does not have eleveted privilege
        returns error if current user does not match <user>.
    
    get [tables | views | indexes | libraries ] for role <role>:
    get [functions | table_mapping_functions | procedures] for role <role>:
    get [privileges | users] for <role>:
      returns objects where <role> has at least one privilege
      if current user does not have eleveted privilege
        returns error if current user has not been granted <role>

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

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

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

    https://github.com/apache/trafodion/pull/1717.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 #1717
    
----
commit afff9935ecd40b0beef8156c773110b40025721f
Author: Roberta Marton <roberta.marton@...>
Date:   2018-09-21T22:29:01Z

    Fixes for TRAFODION-3194 && TRAFODION-3195
    
    TRAFODION-3194 Revoke grant option on objects revokes more that grant option
      changed Privilege Manager to set bitmaps correctly
      removed unused methods from PrivMgrDesc
    
    TRAFODION-3195: Fixes for get commands:
    
    get schemas for user <user>:
      returns schemas owned by the specified user
      if current user does not have elevated privilege,
       returns error if current user does not match <user>.
    
    get schemas for role <role>:
      returns schemas owned by the role,
      if current user does not have elevated privilege,
        returns error if current user has not been granted <role>
    
    get [tables | views | indexes | libraries ] for user <user>:
    get [functions | table_mapping_functions | procedures] for user <user>:
    get [privileges | roles] for user <user>:
      returns objects where <user> has at least one privilege
      if current user does not have eleveted privilege
        returns error if current user does not match <user>.
    
    get [tables | views | indexes | libraries ] for role <role>:
    get [functions | table_mapping_functions | procedures] for role <role>:
    get [privileges | users] for <role>:
      returns objects where <role> has at least one privilege
      if current user does not have eleveted privilege
        returns error if current user has not been granted <role>

----


---

Mime
View raw message