Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.13.0
    • Component/s: Authorization
    • Labels:
      None

      Description

      Implement set roles, show current roles as per functional spec for SQL standard based auth.
      Also authorize set role - check if user belongs to the role.

      1. HIVE-5930.2.patch
        37 kB
        Ashutosh Chauhan
      2. HIVE-5930.1.patch
        38 kB
        Ashutosh Chauhan
      3. HIVE-5930.patch
        37 kB
        Ashutosh Chauhan

        Issue Links

          Activity

          Thejas M Nair created issue -
          Thejas M Nair made changes -
          Field Original Value New Value
          Component/s Authorization [ 12317300 ]
          Thejas M Nair made changes -
          Link This issue is blocked by HIVE-5923 [ HIVE-5923 ]
          Hide
          Thejas M Nair added a comment -

          This sub-task includes parser changes for the same.

          Show
          Thejas M Nair added a comment - This sub-task includes parser changes for the same.
          Thejas M Nair made changes -
          Description Implement set roles, show current roles as per functional spec.
          Implement set roles, show current roles as per functional spec for SQL standard based auth.

          Hide
          Thejas M Nair added a comment -

          We would need to store the current role for the session.

          Show
          Thejas M Nair added a comment - We would need to store the current role for the session.
          Thejas M Nair made changes -
          Original Estimate 12h [ 43200 ]
          Remaining Estimate 12h [ 43200 ]
          Thejas M Nair made changes -
          Original Estimate 12h [ 43200 ] 36h [ 129600 ]
          Remaining Estimate 12h [ 43200 ] 36h [ 129600 ]
          Thejas M Nair made changes -
          Description Implement set roles, show current roles as per functional spec for SQL standard based auth.

          Implement set roles, show current roles as per functional spec for SQL standard based auth.
          Also authorize set role - check if user belongs to the role.
          Hide
          Thejas M Nair added a comment -

          metastore list_roles call can be used to get all roles the user belongs to.

          Show
          Thejas M Nair added a comment - metastore list_roles call can be used to get all roles the user belongs to.
          Ashutosh Chauhan made changes -
          Attachment HIVE-5930.patch [ 12627718 ]
          Ashutosh Chauhan made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Assignee Ashutosh Chauhan [ ashutoshc ]
          Ashutosh Chauhan made changes -
          Remote Link This issue links to "RB request (Web Link)" [ 14109 ]
          Ashutosh Chauhan made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Ashutosh Chauhan added a comment -

          Addressed Brock's comments.

          Show
          Ashutosh Chauhan added a comment - Addressed Brock's comments.
          Ashutosh Chauhan made changes -
          Attachment HIVE-5930.1.patch [ 12627734 ]
          Ashutosh Chauhan made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Ashutosh Chauhan made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Ashutosh Chauhan added a comment -

          Addressed Thejas comments.

          Show
          Ashutosh Chauhan added a comment - Addressed Thejas comments.
          Ashutosh Chauhan made changes -
          Attachment HIVE-5930.2.patch [ 12627769 ]
          Ashutosh Chauhan made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hive QA added a comment -

          Overall: +1 all checks pass

          Here are the results of testing the latest attachment:
          https://issues.apache.org/jira/secure/attachment/12627769/HIVE-5930.2.patch

          SUCCESS: +1 5041 tests passed

          Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/1247/testReport
          Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/1247/console

          Messages:

          Executing org.apache.hive.ptest.execution.PrepPhase
          Executing org.apache.hive.ptest.execution.ExecutionPhase
          Executing org.apache.hive.ptest.execution.ReportingPhase
          

          This message is automatically generated.

          ATTACHMENT ID: 12627769

          Show
          Hive QA added a comment - Overall : +1 all checks pass Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12627769/HIVE-5930.2.patch SUCCESS: +1 5041 tests passed Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/1247/testReport Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/1247/console Messages: Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase This message is automatically generated. ATTACHMENT ID: 12627769
          Hide
          Ashutosh Chauhan added a comment -

          Committed to trunk. Thanks, Thejas for review!

          Show
          Ashutosh Chauhan added a comment - Committed to trunk. Thanks, Thejas for review!
          Ashutosh Chauhan made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Fix Version/s 0.13.0 [ 12324986 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Patch Available Patch Available Open Open
          3h 50m 2 Ashutosh Chauhan 08/Feb/14 02:35
          Open Open Patch Available Patch Available
          66d 1h 17m 3 Ashutosh Chauhan 08/Feb/14 02:36
          Patch Available Patch Available Resolved Resolved
          12h 57m 1 Ashutosh Chauhan 08/Feb/14 15:34

            People

            • Assignee:
              Ashutosh Chauhan
              Reporter:
              Thejas M Nair
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 36h
                36h
                Remaining:
                Remaining Estimate - 36h
                36h
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Development