Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.94.0, 0.95.0
    • Component/s: None
    • Labels:
      None

      Description

      Need to import HADOOP-7384 into our version of Hadoop QA to allow test-patch to be more flexible about patch format

      1. HBASE-5743-2.patch
        3 kB
        Nicolas Spiegelberg
      2. HBASE-5743.patch
        2 kB
        Nicolas Spiegelberg

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK-security #161 (See https://builds.apache.org/job/HBase-TRUNK-security/161/)
          HBASE-5743 Support GIT patches with Hadoop QA (Revision 1310600)

          Result = FAILURE
          nspiegelberg :
          Files :

          • /hbase/trunk/dev-support/smart-apply-patch.sh
          • /hbase/trunk/dev-support/test-patch.sh
          Show
          Hudson added a comment - Integrated in HBase-TRUNK-security #161 (See https://builds.apache.org/job/HBase-TRUNK-security/161/ ) HBASE-5743 Support GIT patches with Hadoop QA (Revision 1310600) Result = FAILURE nspiegelberg : Files : /hbase/trunk/dev-support/smart-apply-patch.sh /hbase/trunk/dev-support/test-patch.sh
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK #2723 (See https://builds.apache.org/job/HBase-TRUNK/2723/)
          HBASE-5743 Support GIT patches with Hadoop QA (Revision 1310600)

          Result = SUCCESS
          nspiegelberg :
          Files :

          • /hbase/trunk/dev-support/smart-apply-patch.sh
          • /hbase/trunk/dev-support/test-patch.sh
          Show
          Hudson added a comment - Integrated in HBase-TRUNK #2723 (See https://builds.apache.org/job/HBase-TRUNK/2723/ ) HBASE-5743 Support GIT patches with Hadoop QA (Revision 1310600) Result = SUCCESS nspiegelberg : Files : /hbase/trunk/dev-support/smart-apply-patch.sh /hbase/trunk/dev-support/test-patch.sh
          Hide
          Todd Lipcon added a comment -

          +1, looks good to me. Make sure you get the +x flag for the new script on commit.

          Show
          Todd Lipcon added a comment - +1, looks good to me. Make sure you get the +x flag for the new script on commit.
          Hide
          Nicolas Spiegelberg added a comment -

          using smart-apply-patch file from hadoop trunk instead of the older version from HADOOP-7384. This runs 'patch' in dry-run mode to determine prefix level, which is a far more fault-tolerant way of solving the problem.

          Show
          Nicolas Spiegelberg added a comment - using smart-apply-patch file from hadoop trunk instead of the older version from HADOOP-7384 . This runs 'patch' in dry-run mode to determine prefix level, which is a far more fault-tolerant way of solving the problem.
          Hide
          Nicolas Spiegelberg added a comment -

          Test plan:

          1. ./dev-support/smart-apply-patch.sh HBASE-5335-trunk-3.patch
          2. git show 5335-trunk | PATCH=echo dev-support/smart-apply-patch.sh -

          Show
          Nicolas Spiegelberg added a comment - Test plan: 1. ./dev-support/smart-apply-patch.sh HBASE-5335 -trunk-3.patch 2. git show 5335-trunk | PATCH=echo dev-support/smart-apply-patch.sh -

            People

            • Assignee:
              Nicolas Spiegelberg
              Reporter:
              Nicolas Spiegelberg
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development