Bigtop
  1. Bigtop
  2. BIGTOP-973

Improve error diagnostics when sed'ing Hadoop version in Oozie package

    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: rpm
    • Labels:
      None

      Description

      There's no good diagnostics when do-component-build dynamically replaced the target Hadoop version. If the "to be replaced" version isn't present in the Oozie pom files, the error might not surfaced until later or at all: mvn will simply pull in incorrect versions of Hadoop libs. case in point: when doing something like this

      sed -i.orig -e "s#2.0.2-alpha#${HADOOP_VERSION}#" ${WORKDIR}/hadooplibs/hadoop-distcp-2/pom.xml
      

      sed will exit successfully even if no replacement took place. We need to guarantee to fail early.

      1. BIGTOP-973.patch
        1 kB
        Konstantin Boudnik
      2. BIGTOP-973.patch
        1 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          33m 2s 1 Konstantin Boudnik 07/May/13 04:44
          Patch Available Patch Available Resolved Resolved
          13h 6m 1 Konstantin Boudnik 07/May/13 17:51
          Resolved Resolved Closed Closed
          45d 6h 58m 1 Roman Shaposhnik 22/Jun/13 00:50
          Roman Shaposhnik made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Konstantin Boudnik made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Konstantin Boudnik added a comment -

          Just committed it as 2dfff51

          Show
          Konstantin Boudnik added a comment - Just committed it as 2dfff51
          Konstantin Boudnik made changes -
          Attachment BIGTOP-973.patch [ 12582120 ]
          Hide
          Konstantin Boudnik added a comment -

          A slightly improved version of the patch that actually terminate the build.

          Show
          Konstantin Boudnik added a comment - A slightly improved version of the patch that actually terminate the build.
          Hide
          Roman Shaposhnik added a comment -

          +1

          Show
          Roman Shaposhnik added a comment - +1
          Hide
          Peter Linnell added a comment -

          +1

          Show
          Peter Linnell added a comment - +1
          Konstantin Boudnik made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Konstantin Boudnik added a comment -

          Also, this fail-early approach will guarantee that we are not missing version update when Oozie is upgraded.

          Show
          Konstantin Boudnik added a comment - Also, this fail-early approach will guarantee that we are not missing version update when Oozie is upgraded.
          Konstantin Boudnik made changes -
          Attachment BIGTOP-973.patch [ 12582029 ]
          Hide
          Konstantin Boudnik added a comment -

          That should do it.

          Show
          Konstantin Boudnik added a comment - That should do it.
          Konstantin Boudnik made changes -
          Field Original Value New Value
          Link This issue relates to BIGTOP-837 [ BIGTOP-837 ]
          Konstantin Boudnik created issue -

            People

            • Assignee:
              Konstantin Boudnik
              Reporter:
              Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development