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

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

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.12.0
    • 0.12.0
    • HCatalog
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: