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

Explicitly say "Hive Internal Error" to ease debugging

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.6.0
    • Fix Version/s: 0.6.0
    • Component/s: None
    • Labels:

      Description

      Our users complain that hive fails error messages like "FAILED: Unknown exception: null".

      We should explicitly mention that's an internal error of Hive, and provide more information (stacktrace) on the screen to ease bug reporting and debugging.

      In other cases, we will still put the detailed information (stacktrace) in the log, since users should be able to figure out what's wrong with a single line of message.

      1. HIVE-1212.1.patch
        43 kB
        Zheng Shao
      2. HIVE-1212.2.patch
        50 kB
        Zheng Shao
      3. HIVE-1212.3.patch
        50 kB
        Zheng Shao

        Activity

        Hide
        zshao Zheng Shao added a comment -

        This also fixes UDFArgumentException reporting.

        Show
        zshao Zheng Shao added a comment - This also fixes UDFArgumentException reporting.
        Hide
        zshao Zheng Shao added a comment -

        Cleaned up some error processing code.

        Show
        zshao Zheng Shao added a comment - Cleaned up some error processing code.
        Hide
        pauly Paul Yang added a comment -

        In QTestUtil.java, there are a few repeated println(error string) calls. Could you either consolidate those to a common string or function call?

        Show
        pauly Paul Yang added a comment - In QTestUtil.java, there are a few repeated println(error string) calls. Could you either consolidate those to a common string or function call?
        Hide
        zshao Zheng Shao added a comment -

        Addressed the comment from Paul.

        Show
        zshao Zheng Shao added a comment - Addressed the comment from Paul.
        Hide
        pauly Paul Yang added a comment -

        Looks good +1

        Show
        pauly Paul Yang added a comment - Looks good +1
        Hide
        nzhang Ning Zhang added a comment -

        OK. I'll test and commit.

        Show
        nzhang Ning Zhang added a comment - OK. I'll test and commit.
        Hide
        nzhang Ning Zhang added a comment -

        Committed. Thanks Zheng!

        Show
        nzhang Ning Zhang added a comment - Committed. Thanks Zheng!

          People

          • Assignee:
            zshao Zheng Shao
            Reporter:
            zshao Zheng Shao
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development