Uploaded image for project: 'HCatalog'
  1. HCatalog
  2. HCATALOG-561

HCatClient should differentiate between "partition not available" and other generic errors.

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.4
    • 0.5, 0.4.1
    • client
    • None

    Description

      In its current form, HCatClient::getPartition() throws the same HCatException for the following cases:
      1. When the client fails to connect to the HCat-server (because of network error or downtime).
      2. When the partition specified by the partition-spec doesn't exist.

      There really should be a distinction between these cases. #1 is an exceptional case that we don't expect to happen often. #2, on the other hand, is a valid return, that's likely to happen. It might make sense to continue to throw exceptions for #1, but return a null (or throw a different, more specific exception) for #2.

      The same holds for HCatClient::listPartitionsByFilter().

      Attachments

        1. HCATALOG-561.branch4.patch
          14 kB
          Mithun Radhakrishnan
        2. HCATALOG-561.rebased.branch4.patch
          14 kB
          Mithun Radhakrishnan
        3. HCATALOG-561.rebased.trunk.patch
          14 kB
          Mithun Radhakrishnan
        4. HCATALOG-561.trunk.patch
          14 kB
          Mithun Radhakrishnan

        Activity

          People

            mithun Mithun Radhakrishnan
            mithun Mithun Radhakrishnan
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: