Hive
  1. Hive
  2. HIVE-366

[hive] testparse depends on a value of a static field

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.3.0
    • Component/s: Testing Infrastructure
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      TestParse depends on the value of "id" which depends on the number of tests run before that

      1. HIVE-366.reopened.2.patch
        219 kB
        Zheng Shao
      2. HIVE-366.reopened.1.patch
        221 kB
        Zheng Shao
      3. hive.366.1.patch
        220 kB
        Namit Jain

        Activity

        Hide
        Prasad Chakka added a comment -

        the patch is to make id a non-bean attribute so it won't be included in the tests. if that is the case then it looks good. +1

        Show
        Prasad Chakka added a comment - the patch is to make id a non-bean attribute so it won't be included in the tests. if that is the case then it looks good. +1
        Hide
        Namit Jain added a comment -

        committed

        Show
        Namit Jain added a comment - committed
        Hide
        Zheng Shao added a comment -

        This problem is back after HIVE-279 is in. Please review this patch which fixes the problem.

        I changed getid() to getIdentifier().

        Show
        Zheng Shao added a comment - This problem is back after HIVE-279 is in. Please review this patch which fixes the problem. I changed getid() to getIdentifier().
        Hide
        Zheng Shao added a comment -

        As the last comment, this problem appeared again after HIVE-279.

        Show
        Zheng Shao added a comment - As the last comment, this problem appeared again after HIVE-279 .
        Hide
        Namit Jain added a comment -

        +1

        looks good - will commit if the tests pass

        Show
        Namit Jain added a comment - +1 looks good - will commit if the tests pass
        Hide
        Zheng Shao added a comment -

        Merged with the recent update in svn.

        Show
        Zheng Shao added a comment - Merged with the recent update in svn.
        Hide
        Namit Jain added a comment -

        committed. Thanks Zheng

        Show
        Namit Jain added a comment - committed. Thanks Zheng

          People

          • Assignee:
            Namit Jain
            Reporter:
            Namit Jain
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development