Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-5413

StorageDelegationAuthorizationProvider uses non-existent org.apache.hive.hcatalog.hbase.HBaseHCatStorageHandler

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.12.0
    • Fix Version/s: 0.12.0
    • Component/s: HCatalog
    • Labels:
      None

      Description

      org.apache.hive.hcatalog.security.StorageDelegationAuthorizationProvider
      has a block like this:

      static

      { registerAuthProvider("org.apache.hadoop.hive.hbase.HBaseStorageHandler", "org.apache.hive.hcatalog.hbase.HBaseAuthorizationProvider"); registerAuthProvider("org.apache.hive.hcatalog.hbase.HBaseHCatStorageHandler", "org.apache.hive.hcatalog.hbase.HBaseAuthorizationProvider"); }

      In reality, HBaseHCatStorageHandler and HBaseAuthorizationProvider only exist in org.apache.hcatalog

      This should be fixed by removing org.apache.hive.hcatalog.security.StorageDelegationAuthorizationProvider. We have org.apache.hcatalog.security.StorageDelegationAuthorizationProvider which is only used by now deprecated hcatalog version of hbase storage handler.

      Also,
      hcatalog/src/test/e2e/hcatalog/tests/pig.conf & hadoop.conf have the same problem.
      In addition, the tests affected in pig.conf/hadoop.conf should use org.apache.hcatalog.pig.HCatLoader/HCatStorer.

      Finally, hadoop.conf#Hadoop_HBase is using org.apache.hive.hcatalog.utils.HBaseReadWrite which internally refers to org.apache.hive.hcatalog.* classes. The later should only use org.apache.hcatalog.* since it's using HBaseHCatStoreageHandler. Also, should move HBaseReadWrite to org.apache.hcatalog for clarity.
      (see the last paragraph of the Description of HIVE-5274)

        Attachments

        1. HIVE-5413.patch
          66 kB
          Eugene Koifman

          Issue Links

            Activity

              People

              • Assignee:
                ekoifman Eugene Koifman
                Reporter:
                ekoifman Eugene Koifman
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: