Hive
  1. Hive
  2. HIVE-2648

Parallel tests fail if master directory is not present

    Details

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

      Description

      Parallel tests should create directories as needed.

      1. ASF.LICENSE.NOT.GRANTED--HIVE-2648.D777.1.patch
        0.5 kB
        Phabricator
      2. ASF.LICENSE.NOT.GRANTED--HIVE-2648.D801.1.patch
        0.5 kB
        Phabricator
      3. HIVE-2648.D777.1.patch
        0.5 kB
        Marek Sapota

        Activity

        Hide
        Ashutosh Chauhan added a comment -

        This issue is closed now. It was released with the fix in 0.9.0. If there is a problem, please open a new jira and link this one with that.

        Show
        Ashutosh Chauhan added a comment - This issue is closed now. It was released with the fix in 0.9.0. If there is a problem, please open a new jira and link this one with that.
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-h0.21 #1141 (See https://builds.apache.org/job/Hive-trunk-h0.21/1141/)
        HIVE-2648 [jira] Parallel tests fail if master directory is not present
        (Marek Sapota via John Sichi)

        Summary:
        Fix the task downloading Ant.

        Parallel tests should create directories as needed.

        Test Plan: EMPTY

        Reviewers: JIRA, jsichi

        Reviewed By: jsichi

        CC: mareksapotafb, jsichi

        Differential Revision: 777

        jvs : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1213447
        Files :

        • /hive/trunk/testutils/ptest/hivetest.py
        Show
        Hudson added a comment - Integrated in Hive-trunk-h0.21 #1141 (See https://builds.apache.org/job/Hive-trunk-h0.21/1141/ ) HIVE-2648 [jira] Parallel tests fail if master directory is not present (Marek Sapota via John Sichi) Summary: Fix the task downloading Ant. Parallel tests should create directories as needed. Test Plan: EMPTY Reviewers: JIRA, jsichi Reviewed By: jsichi CC: mareksapotafb, jsichi Differential Revision: 777 jvs : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1213447 Files : /hive/trunk/testutils/ptest/hivetest.py
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-h0.23.0 #18 (See https://builds.apache.org/job/Hive-trunk-h0.23.0/18/)
        HIVE-2648 [jira] Parallel tests fail if master directory is not present
        (Marek Sapota via John Sichi)

        Summary:
        Fix the task downloading Ant.

        Parallel tests should create directories as needed.

        Test Plan: EMPTY

        Reviewers: JIRA, jsichi

        Reviewed By: jsichi

        CC: mareksapotafb, jsichi

        Differential Revision: 777

        jvs : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1213447
        Files :

        • /hive/trunk/testutils/ptest/hivetest.py
        Show
        Hudson added a comment - Integrated in Hive-trunk-h0.23.0 #18 (See https://builds.apache.org/job/Hive-trunk-h0.23.0/18/ ) HIVE-2648 [jira] Parallel tests fail if master directory is not present (Marek Sapota via John Sichi) Summary: Fix the task downloading Ant. Parallel tests should create directories as needed. Test Plan: EMPTY Reviewers: JIRA, jsichi Reviewed By: jsichi CC: mareksapotafb, jsichi Differential Revision: 777 jvs : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1213447 Files : /hive/trunk/testutils/ptest/hivetest.py
        Hide
        Phabricator added a comment -

        mareksapotafb requested code review of "HIVE-2648 [jira] Parallel tests fail if master directory is not present".
        Reviewers: JIRA

        Add quotes around bash -c to fix commands using cd.

        Parallel tests should create directories as needed.

        TEST PLAN
        EMPTY

        REVISION DETAIL
        https://reviews.facebook.net/D801

        AFFECTED FILES
        testutils/ptest/Ssh.py

        MANAGE HERALD DIFFERENTIAL RULES
        https://reviews.facebook.net/herald/view/differential/

        WHY DID I GET THIS EMAIL?
        https://reviews.facebook.net/herald/transcript/1695/

        Tip: use the X-Herald-Rules header to filter Herald messages in your client.

        Show
        Phabricator added a comment - mareksapotafb requested code review of " HIVE-2648 [jira] Parallel tests fail if master directory is not present". Reviewers: JIRA Add quotes around bash -c to fix commands using cd. Parallel tests should create directories as needed. TEST PLAN EMPTY REVISION DETAIL https://reviews.facebook.net/D801 AFFECTED FILES testutils/ptest/Ssh.py MANAGE HERALD DIFFERENTIAL RULES https://reviews.facebook.net/herald/view/differential/ WHY DID I GET THIS EMAIL? https://reviews.facebook.net/herald/transcript/1695/ Tip: use the X-Herald-Rules header to filter Herald messages in your client.
        Hide
        John Sichi added a comment -

        Committed to trunk, thanks Marek!

        Show
        John Sichi added a comment - Committed to trunk, thanks Marek!
        Hide
        Phabricator added a comment -

        mareksapotafb has committed the revision "HIVE-2648 [jira] Parallel tests fail if master directory is not present".

        REVISION DETAIL
        https://reviews.facebook.net/D777

        COMMIT
        https://reviews.facebook.net/rHIVE1213447

        Show
        Phabricator added a comment - mareksapotafb has committed the revision " HIVE-2648 [jira] Parallel tests fail if master directory is not present". REVISION DETAIL https://reviews.facebook.net/D777 COMMIT https://reviews.facebook.net/rHIVE1213447
        Hide
        Phabricator added a comment -

        jsichi has accepted the revision "HIVE-2648 [jira] Parallel tests fail if master directory is not present".

        +1, committing to trunk.

        REVISION DETAIL
        https://reviews.facebook.net/D777

        Show
        Phabricator added a comment - jsichi has accepted the revision " HIVE-2648 [jira] Parallel tests fail if master directory is not present". +1, committing to trunk. REVISION DETAIL https://reviews.facebook.net/D777
        Hide
        Phabricator added a comment -

        mareksapotafb requested code review of "HIVE-2648 [jira] Parallel tests fail if master directory is not present".
        Reviewers: JIRA

        Fix the task downloading Ant.

        Parallel tests should create directories as needed.

        TEST PLAN
        EMPTY

        REVISION DETAIL
        https://reviews.facebook.net/D777

        AFFECTED FILES
        testutils/ptest/hivetest.py

        MANAGE HERALD DIFFERENTIAL RULES
        https://reviews.facebook.net/herald/view/differential/

        WHY DID I GET THIS EMAIL?
        https://reviews.facebook.net/herald/transcript/1653/

        Tip: use the X-Herald-Rules header to filter Herald messages in your client.

        Show
        Phabricator added a comment - mareksapotafb requested code review of " HIVE-2648 [jira] Parallel tests fail if master directory is not present". Reviewers: JIRA Fix the task downloading Ant. Parallel tests should create directories as needed. TEST PLAN EMPTY REVISION DETAIL https://reviews.facebook.net/D777 AFFECTED FILES testutils/ptest/hivetest.py MANAGE HERALD DIFFERENTIAL RULES https://reviews.facebook.net/herald/view/differential/ WHY DID I GET THIS EMAIL? https://reviews.facebook.net/herald/transcript/1653/ Tip: use the X-Herald-Rules header to filter Herald messages in your client.
        Hide
        Phabricator added a comment -

        mareksapotafb has added reviewers to the revision "HIVE-2648 [jira] Parallel tests fail if master directory is not present".
        Added Reviewers: jsichi

        REVISION DETAIL
        https://reviews.facebook.net/D777

        Show
        Phabricator added a comment - mareksapotafb has added reviewers to the revision " HIVE-2648 [jira] Parallel tests fail if master directory is not present". Added Reviewers: jsichi REVISION DETAIL https://reviews.facebook.net/D777

          People

          • Assignee:
            Marek Sapota
            Reporter:
            Marek Sapota
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development