Hive
  1. Hive
  2. HIVE-1212

Explicitly say "Hive Internal Error" to ease debugging

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major 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.3.patch
        50 kB
        Zheng Shao
      2. HIVE-1212.2.patch
        50 kB
        Zheng Shao
      3. HIVE-1212.1.patch
        43 kB
        Zheng Shao

        Activity

        Hide
        Zheng Shao added a comment -

        This also fixes UDFArgumentException reporting.

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

        Cleaned up some error processing code.

        Show
        Zheng Shao added a comment - Cleaned up some error processing code.
        Hide
        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
        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
        Zheng Shao added a comment -

        Addressed the comment from Paul.

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

        Looks good +1

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

        OK. I'll test and commit.

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

        Committed. Thanks Zheng!

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development