Hive
  1. Hive
  2. HIVE-4741

Add Hive config API to modify the restrict list

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.11.0
    • Fix Version/s: 0.13.0
    • Component/s: None
    • Labels:
      None

      Description

      HiveConf supports a restrict list configuration which is a black list of configurations that can be modified through the 'set x=y'. This is especially useful for HiveServer2 to restrict clients from overriding some of server configurations.
      Currently the restrict list value can't be changed after the HiveConf object is instantiated. It would be useful for custom code like Hooks to have a way to be able to update that setting.

      1. HIVE-4714.2.patch.txt
        6 kB
        Navis
      2. HIVE-4741-1.patch
        6 kB
        Prasad Mujumdar

        Issue Links

          Activity

          Hide
          Prasad Mujumdar added a comment -

          This particular is not directly exposed to end users via SQL or configuration. It fixes the way this restrict list configuration can be changed programatically from a HiveServer2 session. It's only useful for custom hooks.

          Show
          Prasad Mujumdar added a comment - This particular is not directly exposed to end users via SQL or configuration. It fixes the way this restrict list configuration can be changed programatically from a HiveServer2 session. It's only useful for custom hooks.
          Hide
          Lefty Leverenz added a comment -

          What user doc does this need?

          Show
          Lefty Leverenz added a comment - What user doc does this need?
          Hide
          Brock Noland added a comment -

          I have committed this to trunk! Thank you Prasad and Navis!

          Show
          Brock Noland added a comment - I have committed this to trunk! Thank you Prasad and Navis!
          Hide
          Brock Noland added a comment -

          Awesome, thank you Navis!

          +1

          Show
          Brock Noland added a comment - Awesome, thank you Navis! +1
          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/12615539/HIVE-4714.2.patch.txt

          SUCCESS: +1 4687 tests passed

          Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/433/testReport
          Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/433/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: 12615539

          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/12615539/HIVE-4714.2.patch.txt SUCCESS: +1 4687 tests passed Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/433/testReport Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/433/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: 12615539
          Hide
          Navis added a comment -

          Rebased to trunk

          Show
          Navis added a comment - Rebased to trunk
          Hide
          Brock Noland added a comment -

          Prasad Mujumdar, can you rebase this and submit with a name that works for HiveQA?

          Show
          Brock Noland added a comment - Prasad Mujumdar , can you rebase this and submit with a name that works for HiveQA?
          Hide
          Thejas M Nair added a comment -

          Preparing for 0.12 release. Removing fix version of 0.12 for those that are not in 0.12 branch.

          Show
          Thejas M Nair added a comment - Preparing for 0.12 release. Removing fix version of 0.12 for those that are not in 0.12 branch.
          Hide
          Prasad Mujumdar added a comment -
          Show
          Prasad Mujumdar added a comment - Review request on https://reviews.apache.org/r/11905/

            People

            • Assignee:
              Prasad Mujumdar
              Reporter:
              Prasad Mujumdar
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development