Uploaded image for project: 'Phoenix'
  1. Phoenix
  2. PHOENIX-2025

Phoenix-core's hbase-default.xml prevents HBaseTestingUtility from starting up in client apps

    XMLWordPrintableJSON

Details

    • Test
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 4.3.0
    • 4.5.0, 4.4.1
    • None
    • None

    Description

      Phoenix seems to have long had its own version of hbase-default.xml as a test resource in phoenix-core with a single setting to override hbase.defaults.for.version.skip to true. Sometime around Phoenix 4.3, phoenix-core seems to have been split into a main jar and a test jar, and the hbase-default.xml went into the test jar.

      The odd result of this is that in client apps that include the test jar, the classloader in HBaseConfiguration.create() now sees Phoenix's hbase-default.xml, rather than HBase's, and creates a Configuration object without HBase's defaults. One major consequence of this is that the HBaseTestingUtility can't start up, because it relies on those HBase defaults being set. This is a huge problem in a client app that includes the phoenix-core test jar in order to make use of the PhoenixTestDriver and BaseTest classes; the upgrade to 4.3 breaks all tests using the HBaseTestingUtility.

      Attachments

        1. PHOENIX-2025.patch
          8 kB
          Geoffrey Jacoby
        2. PHOENIX-2025_v2.patch
          9 kB
          James R. Taylor
        3. PHOENIX-2025-ClientPortIssue.patch
          3 kB
          Mujtaba Chohan
        4. PHOENIX-2025-default-zkport
          0.9 kB
          maghamravikiran
        5. PHOENIX-2025-optional-client-port.patch
          3 kB
          Thomas D'Silva
        6. PHOENIX-2025-pig-tests.patch
          15 kB
          ravimagham

        Activity

          People

            gjacoby Geoffrey Jacoby
            gjacoby Geoffrey Jacoby
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: