Hive
  1. Hive
  2. HIVE-1297

error message in Hive.checkPaths dumps Java array address instead of path string

    Details

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

      Description

                 if (item.isDir()) {
      -            throw new HiveException("checkPaths: " + src.toString()
      -                + " has nested directory" + item.toString());
      +            throw new HiveException("checkPaths: " + src.getPath()
      +                + " has nested directory " + item.getPath());
                 }
      

        Activity

        Hide
        John Sichi added a comment -

        No test since this is trivial.

        Show
        John Sichi added a comment - No test since this is trivial.
        Hide
        Ning Zhang added a comment -

        looks good. Will commit if tests pass.

        Show
        Ning Zhang added a comment - looks good. Will commit if tests pass.
        Hide
        Ning Zhang added a comment -

        John, TestNegativeCliDriver failed with 35 diffs. Can you take a look?

        Show
        Ning Zhang added a comment - John, TestNegativeCliDriver failed with 35 diffs. Can you take a look?
        Hide
        John Sichi added a comment -

        Were the failures actually related to my change?

        I ran through and everything passed...and the "nested directory" string doesn't even appear in ql/src/test/results/clientnegative/*.out.

        Show
        John Sichi added a comment - Were the failures actually related to my change? I ran through and everything passed...and the "nested directory" string doesn't even appear in ql/src/test/results/clientnegative/*.out.
        Hide
        Ning Zhang added a comment -

        This has already been fixed in another patch.

        Show
        Ning Zhang added a comment - This has already been fixed in another patch.

          People

          • Assignee:
            John Sichi
            Reporter:
            John Sichi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development