Hadoop Common
  1. Hadoop Common
  2. HADOOP-8924

Add maven plugin alternative to shell script to save package-info.java

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0, trunk-win
    • Fix Version/s: 2.1.0-beta
    • Component/s: build
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Target Version/s:

      Description

      Currently, the build process relies on saveVersion.sh to generate package-info.java with a version annotation. The sh binary may not be available on all developers' machines (e.g. Windows without Cygwin). This issue tracks removal of that dependency in Hadoop Common.

      1. HADOOP-8924-branch-2.7.patch
        47 kB
        Chris Nauroth
      2. HADOOP-8924.7.patch
        47 kB
        Chris Nauroth
      3. HADOOP-8924-branch-trunk-win.7.patch
        47 kB
        Chris Nauroth
      4. HADOOP-8924.6.patch
        43 kB
        Chris Nauroth
      5. HADOOP-8924-branch-trunk-win.6.patch
        44 kB
        Chris Nauroth
      6. HADOOP-8924.5.patch
        42 kB
        Chris Nauroth
      7. HADOOP-8924-branch-trunk-win.5.patch
        43 kB
        Chris Nauroth
      8. HADOOP-8924.4.patch
        42 kB
        Chris Nauroth
      9. HADOOP-8924-branch-trunk-win.4.patch
        43 kB
        Chris Nauroth
      10. HADOOP-8924.3.patch
        44 kB
        Chris Nauroth
      11. HADOOP-8924-branch-trunk-win.3.patch
        42 kB
        Chris Nauroth
      12. HADOOP-8924.3.patch
        44 kB
        Chris Nauroth
      13. HADOOP-8924-branch-trunk-win.2.patch
        41 kB
        Chris Nauroth
      14. HADOOP-8924.2.patch
        40 kB
        Chris Nauroth
      15. HADOOP-8924.patch
        25 kB
        Alejandro Abdelnur
      16. HADOOP-8924-branch-trunk-win.patch
        10 kB
        Chris Nauroth

        Issue Links

          Activity

          Chris Nauroth created issue -
          Chris Nauroth made changes -
          Field Original Value New Value
          Summary Hadoop Common creating version annotation must not depend on sh Hadoop Common creating package-info.java must not depend on sh
          Hide
          Chris Nauroth added a comment -

          This patch converts saveVersion.sh to an equivalent saveVersion.py. I also expanded the template and the arguments to include additional fields, so that the same script can be used from both the Hadoop Common and Yarn modules.

          I have tested "mvn generate-sources" in the following combinations:

          1. Mac/git
          2. Windows/git
          3. Windows + Cygwin/git
          4. Mac/svn
          5. Windows/svn
          6. Windows + Cygwin/svn

          All tests produced the correct package-info.java, with the same value for srcChecksum.

          I'm starting this change in branch-trunk-win, because the driver for this change is to get a working Windows build for a branch off of trunk.

          Show
          Chris Nauroth added a comment - This patch converts saveVersion.sh to an equivalent saveVersion.py. I also expanded the template and the arguments to include additional fields, so that the same script can be used from both the Hadoop Common and Yarn modules. I have tested "mvn generate-sources" in the following combinations: 1. Mac/git 2. Windows/git 3. Windows + Cygwin/git 4. Mac/svn 5. Windows/svn 6. Windows + Cygwin/svn All tests produced the correct package-info.java, with the same value for srcChecksum. I'm starting this change in branch-trunk-win, because the driver for this change is to get a working Windows build for a branch off of trunk.
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-trunk-win.patch [ 12548973 ]
          Chris Nauroth made changes -
          Link This issue is required by YARN-158 [ YARN-158 ]
          Hide
          Suresh Srinivas added a comment -

          +1 for the patch. I will commit it to branch-trunk-win.

          Show
          Suresh Srinivas added a comment - +1 for the patch. I will commit it to branch-trunk-win.
          Hide
          Suresh Srinivas added a comment -

          I committed the patch to branch-trunk-win. Thank you Chris.

          Show
          Suresh Srinivas added a comment - I committed the patch to branch-trunk-win. Thank you Chris.
          Suresh Srinivas made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Fix Version/s trunk-win [ 12323306 ]
          Resolution Fixed [ 1 ]
          Hide
          Aaron T. Myers added a comment -

          Hey guys, does it really make sense to trade a dependency on sh for a dependency on python? Maybe it does, but at least on Unix systems I feel like sh is more likely to be available than python. (Honest question here - not trying to be a pain.)

          At the very least, if we stick with this being in python, we should update BUILDING.txt to say that we now have a dependency on python (and perhaps some specific version of python?) in order to build Hadoop.

          Show
          Aaron T. Myers added a comment - Hey guys, does it really make sense to trade a dependency on sh for a dependency on python? Maybe it does, but at least on Unix systems I feel like sh is more likely to be available than python. (Honest question here - not trying to be a pain.) At the very least, if we stick with this being in python, we should update BUILDING.txt to say that we now have a dependency on python (and perhaps some specific version of python?) in order to build Hadoop.
          Hide
          Matt Foley added a comment -

          Hi Aaron, I advised Chris to use Python. As a general matter I think we should stay cross-platform wherever it makes sense to do so. In other words, if something is obviously totally platform-dependent, then go ahead and do two versions, one in shell for Linux, and one in powershell or cmd for Windows. However, where it can be platform-independent, I think we should use a platform-independent scripting language to write a single script (which may include some conditional code for platform-dependent cases if necessary). Obviously it works fine in this case.

          Python is only one possibility, of course. However it seemed a reasonable choice. It's object-oriented, with reasonable IDE support. It's free (like beer and speech), available on essentially all platforms, and in good odour with the OSS community. The only other evident candidate would be Ruby, but I think more people know Python than Ruby, altho I can't substantiate that. (Altho there is the Tiobe Index, which I don't claim is authoritative.)

          This is worth discussing, because as we integrate the Windows port, we'll have lots of opportunity to follow whatever model we agree on. Shall we continue here, or pop it to a common-dev thread?

          And I agree with you that whatever answer we agree on should be documented as a build dependency.

          Show
          Matt Foley added a comment - Hi Aaron, I advised Chris to use Python. As a general matter I think we should stay cross-platform wherever it makes sense to do so. In other words, if something is obviously totally platform-dependent, then go ahead and do two versions, one in shell for Linux, and one in powershell or cmd for Windows. However, where it can be platform-independent, I think we should use a platform-independent scripting language to write a single script (which may include some conditional code for platform-dependent cases if necessary). Obviously it works fine in this case. Python is only one possibility, of course. However it seemed a reasonable choice. It's object-oriented, with reasonable IDE support. It's free (like beer and speech), available on essentially all platforms, and in good odour with the OSS community. The only other evident candidate would be Ruby, but I think more people know Python than Ruby, altho I can't substantiate that. (Altho there is the Tiobe Index , which I don't claim is authoritative.) This is worth discussing, because as we integrate the Windows port, we'll have lots of opportunity to follow whatever model we agree on. Shall we continue here, or pop it to a common-dev thread? And I agree with you that whatever answer we agree on should be documented as a build dependency.
          Hide
          Aaron T. Myers added a comment -

          Hi Matt, a discussion on common-dev@ makes sense to me. I don't feel strongly about what the right answer is, it just wasn't obvious to me that it's python. If it's determined to be python, then that's fine by me.

          And yes, whenever this is resolved, we should definitely file a JIRA to update BUILDING.txt accordingly.

          Show
          Aaron T. Myers added a comment - Hi Matt, a discussion on common-dev@ makes sense to me. I don't feel strongly about what the right answer is, it just wasn't obvious to me that it's python. If it's determined to be python, then that's fine by me. And yes, whenever this is resolved, we should definitely file a JIRA to update BUILDING.txt accordingly.
          Suresh Srinivas made changes -
          Link This issue is related to HADOOP-8420 [ HADOOP-8420 ]
          Hide
          Suresh Srinivas added a comment -

          Related jira from branch-1-win

          Show
          Suresh Srinivas added a comment - Related jira from branch-1-win
          Hide
          Radim Kolar added a comment -

          Its agreed to merge this into trunk? I can prepare and test version of this patch.

          Show
          Radim Kolar added a comment - Its agreed to merge this into trunk? I can prepare and test version of this patch.
          Matt Foley made changes -
          Link This issue relates to HADOOP-9082 [ HADOOP-9082 ]
          Hide
          Matt Foley added a comment -

          After discussion in common-dev@, the community consensus rejected use of Python. (see references in HADOOP-9082) Also, it was encouraged to take functionality out of scripts and move it into core code.

          I suggest that we convert saveVersion.sh to Java, and change the ant script to a recursive call that compiles the java class and then invokes it. Same thing should work in trunk, but be aware there are two instances of saveVersion.sh in trunk: hadoop-common-project/hadoop-common/dev-support/saveVersion.sh and hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh. They are slightly different, but I suspect that can and should be subsumed.

          Show
          Matt Foley added a comment - After discussion in common-dev@, the community consensus rejected use of Python. (see references in HADOOP-9082 ) Also, it was encouraged to take functionality out of scripts and move it into core code. I suggest that we convert saveVersion.sh to Java, and change the ant script to a recursive call that compiles the java class and then invokes it. Same thing should work in trunk, but be aware there are two instances of saveVersion.sh in trunk: hadoop-common-project/hadoop-common/dev-support/saveVersion.sh and hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh. They are slightly different, but I suspect that can and should be subsumed.
          Matt Foley made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Matt Foley made changes -
          Summary Hadoop Common creating package-info.java must not depend on sh Hadoop Common creating package-info.java must not depend on sh, at least for Windows
          Hide
          Chris Nauroth added a comment -

          On branch-trunk-win, when we converted to Python, I consolidated to a single copy of saveVersion.py in /dev-support, which is used by both sub-modules. I needed to introduce some additional arguments to the script so that it could be parameterized differently for hadoop-common vs. hadoop-yarn-common. It didn't add much complexity, and it eliminated the need to maintain 2 different copies of the same code. We could take the same approach to parameterize the equivalent Java code (or whatever else).

          Show
          Chris Nauroth added a comment - On branch-trunk-win, when we converted to Python, I consolidated to a single copy of saveVersion.py in /dev-support, which is used by both sub-modules. I needed to introduce some additional arguments to the script so that it could be parameterized differently for hadoop-common vs. hadoop-yarn-common. It didn't add much complexity, and it eliminated the need to maintain 2 different copies of the same code. We could take the same approach to parameterize the equivalent Java code (or whatever else).
          Hide
          Alejandro Abdelnur added a comment -

          Chris, As I've mentioned in the aliases earlier today, I've got a Maven plugin taking care of saveVersion.sh, I've tested in Windows, OSX, Linux. Would you mind reassigning this JIRA to me and I'll put a patch?

          Thx

          Show
          Alejandro Abdelnur added a comment - Chris, As I've mentioned in the aliases earlier today, I've got a Maven plugin taking care of saveVersion.sh, I've tested in Windows, OSX, Linux. Would you mind reassigning this JIRA to me and I'll put a patch? Thx
          Hide
          Chris Nauroth added a comment -

          Thanks, Alejandro. I've reassigned it to you. I can volunteer to code review it when the patch is available.

          Show
          Chris Nauroth added a comment - Thanks, Alejandro. I've reassigned it to you. I can volunteer to code review it when the patch is available.
          Chris Nauroth made changes -
          Assignee Chris Nauroth [ cnauroth ] Alejandro Abdelnur [ tucu00 ]
          Hide
          Alejandro Abdelnur added a comment -

          Thanks Chris.

          (not setting as patch avail because the plugin is not avail in a Maven repo, details follow).

          Attached is patch that uses a maven plugin to generate the version information.

          The patch changes the VersionInfo class to read all the info from a properties file instead of the package-info.java and the HadoopVersionInfoAnnotation.

          The HadoopVersionInfoAnnotation.java and the saveVersion.sh files are deleted.

          A new properties file 'common-version-info.properties' with ${} variables is available in src/main/resources:

          version=${pom.version}
          revision=${version-info.scm.commit}
          branch=${version-info.scm.branch}
          user=${user.name}
          date=${version-info.build.time}
          url=${version-info.scm.uri}
          srcChecksum=${version-info.source.md5}
          

          This file is copied into target using filtering and the variables are resolved at copy time.

          All the variables that start with version-info are resolved and injected into the project by the new plugin hadoop-maven-plugins using the goal version-info:

          ...
                <plugin>
                  <groupId>org.apache.hadoop.maven.plugin</groupId>
                  <artifactId>hadoop-maven-plugins</artifactId>
                  <executions>
                    <execution>
                      <id>version-info</id>
                      <goals>
                        <goal>version-info</goal>
                      </goals>
                      <configuration>
                        <source>
                          <directory>${basedir}/src/main/java</directory>
                          <includes>
                            <include>**/*.java</include>
                          </includes>
                        </source>
                      </configuration>
                    </execution>
                  </executions>
                </plugin>
          ...
          

          Because the plugin is not yet available, to test it you'll have to build and install it locally. The plugin source is avail at:

          https://github.com/tucu00/hadoop-maven-plugins

          After cloning it, running 'mvn install' will install it locally.

          You'll need to apply HADOOP-9116 before this patch in order to get things to work otherwise the sources JAR ends up in the classpath with the unresolved properties files and it may be first in the classpath.

          I've tested in OSX and Windows and seems to be working as expected.

          I still need to verify the MD5 generated is correct.

          I think the current way VersionInfo works (only getting the md5 of common is not correct), but this is out of scope from this JIRA. Also, the YARN variant seems not be used from the command line when doing 'yarn version'.

          Regarding the maven plugins, we have to figure how we'll build/publish them to a Maven repo in order to use them. I personally think they should be a sub-project of Hadoop to allow us to release new ones independent of Hadoop. Why a Hadoop sub-project and not a general project? IMO, the focus of these plugins (including HADOOP-8887 for cmake and the protoc that is already avail in the GH repo, another JIRA coming soon) is to solve Hadoop build problems.

          Show
          Alejandro Abdelnur added a comment - Thanks Chris. (not setting as patch avail because the plugin is not avail in a Maven repo, details follow). Attached is patch that uses a maven plugin to generate the version information. The patch changes the VersionInfo class to read all the info from a properties file instead of the package-info.java and the HadoopVersionInfoAnnotation. The HadoopVersionInfoAnnotation.java and the saveVersion.sh files are deleted. A new properties file 'common-version-info.properties' with ${} variables is available in src/main/resources: version=${pom.version} revision=${version-info.scm.commit} branch=${version-info.scm.branch} user=${user.name} date=${version-info.build.time} url=${version-info.scm.uri} srcChecksum=${version-info.source.md5} This file is copied into target using filtering and the variables are resolved at copy time. All the variables that start with version-info are resolved and injected into the project by the new plugin hadoop-maven-plugins using the goal version-info: ... <plugin> <groupId>org.apache.hadoop.maven.plugin</groupId> <artifactId>hadoop-maven-plugins</artifactId> <executions> <execution> <id>version-info</id> <goals> <goal>version-info</goal> </goals> <configuration> <source> <directory>${basedir}/src/main/java</directory> <includes> <include>**/*.java</include> </includes> </source> </configuration> </execution> </executions> </plugin> ... Because the plugin is not yet available, to test it you'll have to build and install it locally. The plugin source is avail at: https://github.com/tucu00/hadoop-maven-plugins After cloning it, running 'mvn install' will install it locally. You'll need to apply HADOOP-9116 before this patch in order to get things to work otherwise the sources JAR ends up in the classpath with the unresolved properties files and it may be first in the classpath. I've tested in OSX and Windows and seems to be working as expected. I still need to verify the MD5 generated is correct. I think the current way VersionInfo works (only getting the md5 of common is not correct), but this is out of scope from this JIRA. Also, the YARN variant seems not be used from the command line when doing 'yarn version'. Regarding the maven plugins, we have to figure how we'll build/publish them to a Maven repo in order to use them. I personally think they should be a sub-project of Hadoop to allow us to release new ones independent of Hadoop. Why a Hadoop sub-project and not a general project? IMO, the focus of these plugins (including HADOOP-8887 for cmake and the protoc that is already avail in the GH repo, another JIRA coming soon) is to solve Hadoop build problems.
          Alejandro Abdelnur made changes -
          Attachment HADOOP-8924.patch [ 12556055 ]
          Alejandro Abdelnur made changes -
          Link This issue depends on HADOOP-9116 [ HADOOP-9116 ]
          Alejandro Abdelnur made changes -
          Link This issue blocks HADOOP-9117 [ HADOOP-9117 ]
          Hide
          Alejandro Abdelnur added a comment -

          Along similar lines, I've created and put a patch for protoc using a maven plugin, HADOOP-9117.

          Show
          Alejandro Abdelnur added a comment - Along similar lines, I've created and put a patch for protoc using a maven plugin, HADOOP-9117 .
          Hide
          Matt Foley added a comment -

          -1. Alejandro, I've said in several threads that this level of complexity is not acceptable for this task.

          We have no need to use maven plugins everywhere they could possibly be applied, when a simpler approach will do. Very few people in the community are expert with these plug-ins, and this approach is increasing the opacity, not the simplicity. HADOOP-8887 is a reasonable attempt to do something already complex (native builds) in a way that fits better into the build process. But HADOOP-8887 has been in process for two months, and apparently is still not ready for prime time. Capturing the version information should be simple, and this approach isn't simple at all. Furthermore, the underlying plugin (referenced from github) is 3 files with 374 source loc, not simple and transparent.

          Show
          Matt Foley added a comment - -1. Alejandro, I've said in several threads that this level of complexity is not acceptable for this task. We have no need to use maven plugins everywhere they could possibly be applied, when a simpler approach will do. Very few people in the community are expert with these plug-ins, and this approach is increasing the opacity, not the simplicity. HADOOP-8887 is a reasonable attempt to do something already complex (native builds) in a way that fits better into the build process. But HADOOP-8887 has been in process for two months, and apparently is still not ready for prime time. Capturing the version information should be simple, and this approach isn't simple at all. Furthermore, the underlying plugin (referenced from github) is 3 files with 374 source loc, not simple and transparent.
          Hide
          Radim Kolar added a comment -

          I also do not like custom maven plugin. Can't it be reworked into separate java program?

          Show
          Radim Kolar added a comment - I also do not like custom maven plugin. Can't it be reworked into separate java program?
          Hide
          Alejandro Abdelnur added a comment -

          If I'm not mistaken the votes for using Maven plugins didn't have a single -1s, including a +1 from you Randim.

          Writing Maven plugins is more complex that writing scripts, I don't dispute this. The main motivations for using Maven plugins is to keep things in the POM declarative and to hide (if necessary) different handlings for different platforms.

          IMO, having a bunch of stand alone java programs that do this will have the same complexity than a Maven plugin, plus we'll lose all the benefits from Maven, version management, distribution/download via maven repo, declarative builds.

          Regarding HADOOP-8887, it has been sitting around for a while, not because it is incomplete or complex but because we are not solving how the plugin gets into a Maven repo so it is available for the Hadoop build just to download it. I thinking we should have a discussion in common-dev@ on how to address this.

          Going back to this particular JIRA, the other approach is to do it in the way Oozie does it:

          For building a release tarball, there is a script 'mkdistro' which computes the SCM (SVN or GIT) info (uri, branch, commit), the build timestamp and passes it the Maven invocation as -D options, then they are avail as properties in the POM and the version-info.properties file gets filtered with those.

          Essentially is this patch minus the plugin plus a script to call Maven.

          The drawback of this approach is that the script has to do very much what saveVersion.sh does, meaning we are at square one.

          We never tackled this in Oozie as a plugin because, so far, nobody as attempted to build/run Oozie server in Windows. I assume in Oozie we'd go the plugin way when such need arises.

          I'm OK in updating this patch to do things how Oozie does it today, using bash. For how to do that in Windows command without using cygwin, I don't know. Again, I think we are back to square one with this approach, we are just the script logic of saveVersion.sh around.

          Show
          Alejandro Abdelnur added a comment - If I'm not mistaken the votes for using Maven plugins didn't have a single -1s, including a +1 from you Randim. Writing Maven plugins is more complex that writing scripts, I don't dispute this. The main motivations for using Maven plugins is to keep things in the POM declarative and to hide (if necessary) different handlings for different platforms. IMO, having a bunch of stand alone java programs that do this will have the same complexity than a Maven plugin, plus we'll lose all the benefits from Maven, version management, distribution/download via maven repo, declarative builds. Regarding HADOOP-8887 , it has been sitting around for a while, not because it is incomplete or complex but because we are not solving how the plugin gets into a Maven repo so it is available for the Hadoop build just to download it. I thinking we should have a discussion in common-dev@ on how to address this. Going back to this particular JIRA, the other approach is to do it in the way Oozie does it: For building a release tarball, there is a script 'mkdistro' which computes the SCM (SVN or GIT) info (uri, branch, commit), the build timestamp and passes it the Maven invocation as -D options, then they are avail as properties in the POM and the version-info.properties file gets filtered with those. Essentially is this patch minus the plugin plus a script to call Maven. The drawback of this approach is that the script has to do very much what saveVersion.sh does, meaning we are at square one. We never tackled this in Oozie as a plugin because, so far, nobody as attempted to build/run Oozie server in Windows. I assume in Oozie we'd go the plugin way when such need arises. I'm OK in updating this patch to do things how Oozie does it today, using bash. For how to do that in Windows command without using cygwin, I don't know. Again, I think we are back to square one with this approach, we are just the script logic of saveVersion.sh around.
          Hide
          Radim Kolar added a comment -

          I voted for Groovy or some other language in pom.xml, but Matt is -1 on groovy in maven.

          My understanding of 2nd vote option was to use an existing maven plugin and combine it with some language which JVM provides. There is couple of options available.

          I offered to rework python code into Groovy in maven, because its high priority for me to get builds working on windows. I can rework it into pretty much anything just you guys need to decide what direction you want to go.

          Show
          Radim Kolar added a comment - I voted for Groovy or some other language in pom.xml, but Matt is -1 on groovy in maven. My understanding of 2nd vote option was to use an existing maven plugin and combine it with some language which JVM provides. There is couple of options available. I offered to rework python code into Groovy in maven, because its high priority for me to get builds working on windows. I can rework it into pretty much anything just you guys need to decide what direction you want to go.
          Hide
          Matt Foley added a comment -

          How about the following:

          1. convert saveVersion.sh into saveVersion.java. Pass the parameter values (including the root of the source tree for checksum calculation) in to the class constructor at invocation time.
          2. add a dependency rule that causes saveVersion.class to be compiled before any other java is compiled, and separately from it.
          3. add a simple execution outcall from ant or maven that calls the class, and passes the parameter values in to the call. If maven requires a completely vanilla "plug-in" to do this well, I'm okay with that. I just don't want the significant functionality of saveVersion.java to be subsumed into the plug-in and become opaque to the hadoop source.

          Show
          Matt Foley added a comment - How about the following: 1. convert saveVersion.sh into saveVersion.java. Pass the parameter values (including the root of the source tree for checksum calculation) in to the class constructor at invocation time. 2. add a dependency rule that causes saveVersion.class to be compiled before any other java is compiled, and separately from it. 3. add a simple execution outcall from ant or maven that calls the class, and passes the parameter values in to the call. If maven requires a completely vanilla "plug-in" to do this well, I'm okay with that. I just don't want the significant functionality of saveVersion.java to be subsumed into the plug-in and become opaque to the hadoop source.
          Hide
          Radim Kolar added a comment -

          it might be workable. Store saveVersion.java into separate directory but same module as source files, then bind maven-compiler and maven-exec plugins to generate-sources or process-sources phase.

          Show
          Radim Kolar added a comment - it might be workable. Store saveVersion.java into separate directory but same module as source files, then bind maven-compiler and maven-exec plugins to generate-sources or process-sources phase.
          Hide
          Radim Kolar added a comment -

          My solution will end with 2 copies of that code in hadoop tree. Without writing hadoop plugin or using scripting language, i do not know better way.

          Show
          Radim Kolar added a comment - My solution will end with 2 copies of that code in hadoop tree. Without writing hadoop plugin or using scripting language, i do not know better way.
          Hide
          Chris Nauroth added a comment -

          The discussion on common-dev mentioned the idea of using a Maven plugin, but building it within the reactor as a new sub-module of the main Hadoop project instead of a separately deployed artifact. It was mentioned that Avro already does this kind of thing.

          I wanted to see what this would look like, so I took the Maven plugin code from Alejandro's github repo, copied it into the hadoop-common tree, and adjusted the pom.xml files to reference this. (I did not include ProtocMojo.java, since that isn't the scope of this jira.) I'm attaching patches for trunk and branch-trunk-win that show this work.

          I'm hopeful that this approach will be an acceptable compromise for everyone. The version info code is still part of the main Hadoop project, easily visible to any Hadoop developer or release engineer. The Maven plugin framework does require some extra boilerplate code, but the focus is still on generating the version info. By making it a plugin instead of a stand-alone executable, we avoid the need to duplicate the code for common and yarn, like Radim mentioned.

          Show
          Chris Nauroth added a comment - The discussion on common-dev mentioned the idea of using a Maven plugin, but building it within the reactor as a new sub-module of the main Hadoop project instead of a separately deployed artifact. It was mentioned that Avro already does this kind of thing. I wanted to see what this would look like, so I took the Maven plugin code from Alejandro's github repo, copied it into the hadoop-common tree, and adjusted the pom.xml files to reference this. (I did not include ProtocMojo.java, since that isn't the scope of this jira.) I'm attaching patches for trunk and branch-trunk-win that show this work. I'm hopeful that this approach will be an acceptable compromise for everyone. The version info code is still part of the main Hadoop project, easily visible to any Hadoop developer or release engineer. The Maven plugin framework does require some extra boilerplate code, but the focus is still on generating the version info. By making it a plugin instead of a stand-alone executable, we avoid the need to duplicate the code for common and yarn, like Radim mentioned.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.2.patch [ 12564054 ]
          Attachment HADOOP-8924-branch-trunk-win.2.patch [ 12564055 ]
          Hide
          Alejandro Abdelnur added a comment -

          Chris, thanks for taking up on this. LGTM, submitting patch for test-patch to pick it up.

          Matt, is this variant of the approach OK with you? If so, would you withdraw your -1?

          Show
          Alejandro Abdelnur added a comment - Chris, thanks for taking up on this. LGTM, submitting patch for test-patch to pick it up. Matt, is this variant of the approach OK with you? If so, would you withdraw your -1?
          Hide
          Matt Foley added a comment -

          Yes, this looks okay, and thanks for doing this, Chris. The "real work" is being done in Java [ack Alejandro for the first version of that], which is goodness and also now a transparent part of the Hadoop distribution. And invoking it from a Maven plugin is also [I'll admit :-)] goodness, and hopefully makes this approach acceptable to you, Alejandro?

          Chris, please test against RHEL5, RHEL6, and Windows. Hopefully the MD5's will be the same for all three platforms, and the same as the saveVersion.sh script produces. (Achieving those constraints was the tricky part of the python version.) Thanks.

          Show
          Matt Foley added a comment - Yes, this looks okay, and thanks for doing this, Chris. The "real work" is being done in Java [ack Alejandro for the first version of that] , which is goodness and also now a transparent part of the Hadoop distribution. And invoking it from a Maven plugin is also [I'll admit :-)] goodness, and hopefully makes this approach acceptable to you, Alejandro? Chris, please test against RHEL5, RHEL6, and Windows. Hopefully the MD5's will be the same for all three platforms, and the same as the saveVersion.sh script produces. (Achieving those constraints was the tricky part of the python version.) Thanks.
          Hide
          Matt Foley added a comment -

          +1, conditional on those tests, and test-patch results.

          Show
          Matt Foley added a comment - +1, conditional on those tests, and test-patch results.
          Hide
          Alejandro Abdelnur added a comment -

          Yeah, I'm good with this approach.

          The MD5 computed by saveVersion.sh is:

          srcChecksum=`find src/main/java -name '*.java' | LC_ALL=C sort | xargs md5sum | md5sum | cut -d ' ' -f 1`
          

          This is, sort the files, MD5 on each file, and them MD5 on the MD5 output of all files.

          The MD5 computed by the plugin is a single MD5 on the content of ALL files, sorted.

          So the MD5s computed by the script and the plugin won't be the same.

          But the MD5 of the plugin should always be the same in diff platforms for the same source.

          Show
          Alejandro Abdelnur added a comment - Yeah, I'm good with this approach. The MD5 computed by saveVersion.sh is: srcChecksum=`find src/main/java -name '*.java' | LC_ALL=C sort | xargs md5sum | md5sum | cut -d ' ' -f 1` This is, sort the files, MD5 on each file, and them MD5 on the MD5 output of all files. The MD5 computed by the plugin is a single MD5 on the content of ALL files, sorted. So the MD5s computed by the script and the plugin won't be the same. But the MD5 of the plugin should always be the same in diff platforms for the same source.
          Hide
          Chris Nauroth added a comment -

          Thanks, Matt and Alejandro. I'm seeing differences between MD5s on Mac vs. Windows due to file sorting differences. I'm working on an updated patch to address it.

          One other difference between this and saveVersion.sh is that saveVersion.sh included generated-sources, but this plugin won't, because we're binding it to the initialize phase. During the Python port, we chose to exclude generated-sources, because we can't guarantee that generated stuff (i.e. protoc) will generate code with the same line endings regardless of platform. This had been causing different MD5s on Windows, so we changed the Python script to skip generated-sources. I think skipping generated-sources is fine. I'm just mentioning it here so that everyone is aware of the difference.

          Matt, I was planning on testing Mac, Windows, and Ubuntu, because that's what I have access to right now. Is there any particular reason to retest on RHEL5 and RHEL6 in addition to that? That probably would have been important for the Python script, because RHEL distributions are tightly coupled to a particular Python version, but I don't think it's relevant for the Java implementation. If you disagree, let me know, and I'll spin up some RHEL VMs.

          Show
          Chris Nauroth added a comment - Thanks, Matt and Alejandro. I'm seeing differences between MD5s on Mac vs. Windows due to file sorting differences. I'm working on an updated patch to address it. One other difference between this and saveVersion.sh is that saveVersion.sh included generated-sources, but this plugin won't, because we're binding it to the initialize phase. During the Python port, we chose to exclude generated-sources, because we can't guarantee that generated stuff (i.e. protoc) will generate code with the same line endings regardless of platform. This had been causing different MD5s on Windows, so we changed the Python script to skip generated-sources. I think skipping generated-sources is fine. I'm just mentioning it here so that everyone is aware of the difference. Matt, I was planning on testing Mac, Windows, and Ubuntu, because that's what I have access to right now. Is there any particular reason to retest on RHEL5 and RHEL6 in addition to that? That probably would have been important for the Python script, because RHEL distributions are tightly coupled to a particular Python version, but I don't think it's relevant for the Java implementation. If you disagree, let me know, and I'll spin up some RHEL VMs.
          Hide
          Alejandro Abdelnur added a comment -

          on generated files, I assume you refer to the protoc files, we could add the protoc sources the MD5.

          Show
          Alejandro Abdelnur added a comment - on generated files, I assume you refer to the protoc files, we could add the protoc sources the MD5.
          Hide
          Chris Nauroth added a comment -

          Line endings on the .proto files were consistent across platforms. The .java files generated by protoc had Windows line endings though.

          I'll add *.proto to the fileset in my next version of the patch.

          Show
          Chris Nauroth added a comment - Line endings on the .proto files were consistent across platforms. The .java files generated by protoc had Windows line endings though. I'll add *.proto to the fileset in my next version of the patch.
          Hide
          Chris Nauroth added a comment -

          I'm attaching version 3 of the patch for trunk and branch-trunk-win. The differences since last time are that *.proto is included in the fileset for the MD5 calculation (see pom.xml files), and the MD5 calculation uses a platform-independent sort order for processing the files (see VersionInfoMojo#computeMD5). The sort logic is a port of the earlier Python code on branch-trunk-win.

          I've tested this on Mac, Windows, and Ubuntu. MD5 calculations are consistent across platforms (though intentionally different from saveVersion.sh for the reasons discussed above).

          Show
          Chris Nauroth added a comment - I'm attaching version 3 of the patch for trunk and branch-trunk-win. The differences since last time are that *.proto is included in the fileset for the MD5 calculation (see pom.xml files), and the MD5 calculation uses a platform-independent sort order for processing the files (see VersionInfoMojo#computeMD5 ). The sort logic is a port of the earlier Python code on branch-trunk-win. I've tested this on Mac, Windows, and Ubuntu. MD5 calculations are consistent across platforms (though intentionally different from saveVersion.sh for the reasons discussed above).
          Chris Nauroth made changes -
          Attachment HADOOP-8924.3.patch [ 12564481 ]
          Attachment HADOOP-8924-branch-trunk-win.3.patch [ 12564482 ]
          Chris Nauroth made changes -
          Status Reopened [ 4 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12564482/HADOOP-8924-branch-trunk-win.3.patch
          against trunk revision .

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2029//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12564482/HADOOP-8924-branch-trunk-win.3.patch against trunk revision . -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2029//console This message is automatically generated.
          Hide
          Chris Nauroth added a comment -

          Re-uploading trunk patch so that Jenkins picks up the right file.

          Show
          Chris Nauroth added a comment - Re-uploading trunk patch so that Jenkins picks up the right file.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.3.patch [ 12564490 ]
          Hide
          Alejandro Abdelnur added a comment -

          patch LGTM (pending Jenkins), but somebody else should +1 as I'm partially responsible for it.

          One thing I've forgot to mention is that currently (and with this patch) MD5 are done only for the sources in common and in yarn. And the VersionInfo from common is used in hdfs. IMO, we should either have a global MD5 & VersionInfo for the whole project or one per module. This is out of scope of this JIRA, just wanted to bring it up.

          Show
          Alejandro Abdelnur added a comment - patch LGTM (pending Jenkins), but somebody else should +1 as I'm partially responsible for it. One thing I've forgot to mention is that currently (and with this patch) MD5 are done only for the sources in common and in yarn. And the VersionInfo from common is used in hdfs. IMO, we should either have a global MD5 & VersionInfo for the whole project or one per module. This is out of scope of this JIRA, just wanted to bring it up.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12564490/HADOOP-8924.3.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          -1 javadoc. The javadoc tool appears to have generated 2 warning messages.

          -1 eclipse:eclipse. The patch failed to build with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-assemblies hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common:

          org.apache.hadoop.ha.TestZKFailoverController

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2030//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2030//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12564490/HADOOP-8924.3.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. -1 javadoc . The javadoc tool appears to have generated 2 warning messages. -1 eclipse:eclipse . The patch failed to build with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-assemblies hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common: org.apache.hadoop.ha.TestZKFailoverController +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2030//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2030//console This message is automatically generated.
          Chris Nauroth made changes -
          Link This issue requires HADOOP-9202 [ HADOOP-9202 ]
          Chris Nauroth made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-trunk-win.4.patch [ 12564578 ]
          Hide
          Chris Nauroth added a comment -

          I'm attaching version 4 of the patch. Jenkins reported new javadoc warnings. This was caused by the non-standard doclet tags like @goal used by Maven. To resolve this, I've switched the code to use Maven's plugin annotations instead of doclet tags.

          Additionally, Jenkins reported a failure for this patch during mvn eclipse:eclipse. This is actually a bug in test-patch.sh. I've submitted a separate patch to address that in HADOOP-9202. The HADOOP-9202 patch must be committed first before we can commit HADOOP-8924, so I would appreciate a code review on that one too. It's a small diff that just reorders some of the build steps.

          Jenkins also gave -1 for no new tests. I think this is acceptable for a piece of build scripting, but if you disagree, let me know, and I can work on some tests.

          Show
          Chris Nauroth added a comment - I'm attaching version 4 of the patch. Jenkins reported new javadoc warnings. This was caused by the non-standard doclet tags like @goal used by Maven. To resolve this, I've switched the code to use Maven's plugin annotations instead of doclet tags. Additionally, Jenkins reported a failure for this patch during mvn eclipse:eclipse. This is actually a bug in test-patch.sh. I've submitted a separate patch to address that in HADOOP-9202 . The HADOOP-9202 patch must be committed first before we can commit HADOOP-8924 , so I would appreciate a code review on that one too. It's a small diff that just reorders some of the build steps. Jenkins also gave -1 for no new tests. I think this is acceptable for a piece of build scripting, but if you disagree, let me know, and I can work on some tests.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.4.patch [ 12564579 ]
          Hide
          Chris Nauroth added a comment -

          HADOOP-9202 just got committed. (Thank you, Robert Joseph Evans.) I'm resubmitting the v4 patch to Jenkins.

          Show
          Chris Nauroth added a comment - HADOOP-9202 just got committed. (Thank you, Robert Joseph Evans .) I'm resubmitting the v4 patch to Jenkins.
          Chris Nauroth made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12564579/HADOOP-8924.4.patch
          against trunk revision .

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2039//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12564579/HADOOP-8924.4.patch against trunk revision . -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2039//console This message is automatically generated.
          Chris Nauroth made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-trunk-win.5.patch [ 12564736 ]
          Hide
          Chris Nauroth added a comment -

          I'm attaching version 5 of the patch to rebase it against recent trunk changes.

          Show
          Chris Nauroth added a comment - I'm attaching version 5 of the patch to rebase it against recent trunk changes.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.5.patch [ 12564737 ]
          Chris Nauroth made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12564737/HADOOP-8924.5.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          -1 release audit. The applied patch generated 2 release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common:

          org.apache.hadoop.ha.TestZKFailoverController

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//testReport/
          Release audit warnings: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//artifact/trunk/patchprocess/patchReleaseAuditProblems.txt
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12564737/HADOOP-8924.5.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. -1 release audit . The applied patch generated 2 release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common: org.apache.hadoop.ha.TestZKFailoverController +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//artifact/trunk/patchprocess/patchReleaseAuditProblems.txt Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2040//console This message is automatically generated.
          Hide
          Chris Nauroth added a comment -

          The release audit failures are unrelated to this patch:

           !????? /home/jenkins/jenkins-slave/workspace/PreCommit-HADOOP-Build/trunk/hadoop-hdfs-project/hadoop-hdfs/src/site/resources/images/hdfsdatanodes.odg
           !????? /home/jenkins/jenkins-slave/workspace/PreCommit-HADOOP-Build/trunk/hadoop-hdfs-project/hadoop-hdfs/src/site/resources/images/hdfsarchitecture.odg
          Lines that start with ????? in the release audit report indicate files that do not have an Apache license header.
          

          It seems like rat can't figure out that these are binary files. I haven't been able to repro the problem on any of my own machines. This seems to be causing a problem for some other patches too.

          The failure in TestZKFailoverController is unrelated. This test has been flaky lately, failing on a few other patches.

          Show
          Chris Nauroth added a comment - The release audit failures are unrelated to this patch: !????? /home/jenkins/jenkins-slave/workspace/PreCommit-HADOOP-Build/trunk/hadoop-hdfs-project/hadoop-hdfs/src/site/resources/images/hdfsdatanodes.odg !????? /home/jenkins/jenkins-slave/workspace/PreCommit-HADOOP-Build/trunk/hadoop-hdfs-project/hadoop-hdfs/src/site/resources/images/hdfsarchitecture.odg Lines that start with ????? in the release audit report indicate files that do not have an Apache license header. It seems like rat can't figure out that these are binary files. I haven't been able to repro the problem on any of my own machines. This seems to be causing a problem for some other patches too. The failure in TestZKFailoverController is unrelated. This test has been flaky lately, failing on a few other patches.
          Hide
          Matt Foley added a comment -

          Hi Chris and Alejandro, it's fine with me to test on Mac, Win, and Ubuntu, for consistent MD5, and I agree it isn't strictly necessary to have the same MD5 as saveVersion.sh.

          But I'm concerned about:

          One thing I've forgot to mention is that currently (and with this patch) MD5 are done only for the sources in common and in yarn. And the VersionInfo from common is used in hdfs. IMO, we should either have a global MD5 & VersionInfo for the whole project or one per module. This is out of scope of this JIRA, just wanted to bring it up.

          I didn't notice because I was focusing on the Hadoop-1 version where I'm more familiar with the env. In Hadoop-1 there is simply one checksum for the whole project.

          Absent that, I think the checksum for hdfs in Hadoop-2 should be created by summing the MD5 for the hdfs sub-project sources, so each sub-project sums its own sources (as do common and yarn). The most important use of the checksum is to enforce the constraint that all the servers talking to each other be running code compiled from the same source tree. And that is clearly important to HDFS, and won't be enforced with the current scheme.

          Agree if this behavior was already in the Hadoop-2 code we don't have to fix it here, but if it would be a simple change, I would support fixing it here. If not, please open a bug for it. Thanks.

          Show
          Matt Foley added a comment - Hi Chris and Alejandro, it's fine with me to test on Mac, Win, and Ubuntu, for consistent MD5, and I agree it isn't strictly necessary to have the same MD5 as saveVersion.sh. But I'm concerned about: One thing I've forgot to mention is that currently (and with this patch) MD5 are done only for the sources in common and in yarn. And the VersionInfo from common is used in hdfs. IMO, we should either have a global MD5 & VersionInfo for the whole project or one per module. This is out of scope of this JIRA, just wanted to bring it up. I didn't notice because I was focusing on the Hadoop-1 version where I'm more familiar with the env. In Hadoop-1 there is simply one checksum for the whole project. Absent that, I think the checksum for hdfs in Hadoop-2 should be created by summing the MD5 for the hdfs sub-project sources, so each sub-project sums its own sources (as do common and yarn). The most important use of the checksum is to enforce the constraint that all the servers talking to each other be running code compiled from the same source tree. And that is clearly important to HDFS, and won't be enforced with the current scheme. Agree if this behavior was already in the Hadoop-2 code we don't have to fix it here, but if it would be a simple change, I would support fixing it here. If not, please open a bug for it. Thanks.
          Chris Nauroth made changes -
          Link This issue is related to HADOOP-9207 [ HADOOP-9207 ]
          Hide
          Chris Nauroth added a comment -

          Thanks, Matt. I just created HADOOP-9207 to address it separately.

          The most important use of the checksum is to enforce the constraint that all the servers talking to each other be running code compiled from the same source tree.

          Considering this, I think the best solution is to calculate a single global checksum for the whole project and use the same value across all modules. That's going to take some restructuring, and I'd prefer to address it in a separate jira instead of holding up this one.

          Alejandro, I think this is all set as long as you are OK with the version 5 patch and my justifications on the tests and release audit warnings. Let me know what you think.

          Show
          Chris Nauroth added a comment - Thanks, Matt. I just created HADOOP-9207 to address it separately. The most important use of the checksum is to enforce the constraint that all the servers talking to each other be running code compiled from the same source tree. Considering this, I think the best solution is to calculate a single global checksum for the whole project and use the same value across all modules. That's going to take some restructuring, and I'd prefer to address it in a separate jira instead of holding up this one. Alejandro, I think this is all set as long as you are OK with the version 5 patch and my justifications on the tests and release audit warnings. Let me know what you think.
          Hide
          Alejandro Abdelnur added a comment -

          Chris, latest patch LGTM, but again, somebody else should review it as I've written part of it. Thx. Also, I've just commented in HADOOP-9207 how we could do the checksum for ALL source.

          Show
          Alejandro Abdelnur added a comment - Chris, latest patch LGTM, but again, somebody else should review it as I've written part of it. Thx. Also, I've just commented in HADOOP-9207 how we could do the checksum for ALL source.
          Hide
          Alejandro Abdelnur added a comment -

          Finally, I think we should have an additional goal in print-version-info that prints the computed MD5, this would help somebody to easily obtain the checksum of a source and verify a build.

          Show
          Alejandro Abdelnur added a comment - Finally, I think we should have an additional goal in print-version-info that prints the computed MD5, this would help somebody to easily obtain the checksum of a source and verify a build.
          Hide
          Suresh Srinivas added a comment -

          Looking at the code VersionInfo.java code looks like it can be improved. It has _get* instance methods and get* static methods. These static methods are hidden by other classes that extend this class. Is it not possible to use one implementation of VersionInfo? All that I see different between variants of this class is how they decide on the properties file to read. So instead of passing component name in the constructor, can the properties name be passed?

          In pom.xml file, the change where in resources *-version-info.properties seems to be both included and excluded. Adding a brief comment on why this is being done will help understanding.

          Show
          Suresh Srinivas added a comment - Looking at the code VersionInfo.java code looks like it can be improved. It has _get* instance methods and get* static methods. These static methods are hidden by other classes that extend this class. Is it not possible to use one implementation of VersionInfo? All that I see different between variants of this class is how they decide on the properties file to read. So instead of passing component name in the constructor, can the properties name be passed? In pom.xml file, the change where in resources *-version-info.properties seems to be both included and excluded. Adding a brief comment on why this is being done will help understanding.
          Hide
          Suresh Srinivas added a comment -

          One other comment - please add javadoc for the newly added classes. For example, it is missing for Excec, FileSetUtils etc.

          Show
          Suresh Srinivas added a comment - One other comment - please add javadoc for the newly added classes. For example, it is missing for Excec , FileSetUtils etc.
          Hide
          Chris Nauroth added a comment -

          We can consolidate to a single VersionInfo class and delete YarnVersionInfo in the scope of HADOOP-9207, which will convert the build to calculate a single checksum across the whole repository.

          I'll upload a new patch shortly with the comment added to pom.xml.

          Show
          Chris Nauroth added a comment - We can consolidate to a single VersionInfo class and delete YarnVersionInfo in the scope of HADOOP-9207 , which will convert the build to calculate a single checksum across the whole repository. I'll upload a new patch shortly with the comment added to pom.xml.
          Hide
          Alejandro Abdelnur added a comment -

          Yes, we can simplify the VersionInfo.java if we get rid of the YARN one. Regarding the exclude/include of the props file in the POM, is to do copy with filtering.

          Show
          Alejandro Abdelnur added a comment - Yes, we can simplify the VersionInfo.java if we get rid of the YARN one. Regarding the exclude/include of the props file in the POM, is to do copy with filtering.
          Chris Nauroth made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-trunk-win.6.patch [ 12565526 ]
          Hide
          Chris Nauroth added a comment -

          Attaching version 6 of the patch. The only difference from the prior version is that I added comments to pom.xml for hadoop-common and hadoop-yarn-common explaining the resource exclusion/inclusion with property substitution.

          Show
          Chris Nauroth added a comment - Attaching version 6 of the patch. The only difference from the prior version is that I added comments to pom.xml for hadoop-common and hadoop-yarn-common explaining the resource exclusion/inclusion with property substitution.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.6.patch [ 12565527 ]
          Hide
          Chris Nauroth added a comment -

          Sorry, I missed Suresh's comment about javadocs before uploading that last patch. Thanks for the feedback, Suresh. I'll do that now.

          Show
          Chris Nauroth added a comment - Sorry, I missed Suresh's comment about javadocs before uploading that last patch. Thanks for the feedback, Suresh. I'll do that now.
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-trunk-win.7.patch [ 12565538 ]
          Hide
          Chris Nauroth added a comment -

          Attaching version 7 of the patch for trunk and branch-trunk-win. This adds the pom.xml comments and more javadocs for the Maven plugin classes.

          Show
          Chris Nauroth added a comment - Attaching version 7 of the patch for trunk and branch-trunk-win. This adds the pom.xml comments and more javadocs for the Maven plugin classes.
          Chris Nauroth made changes -
          Attachment HADOOP-8924.7.patch [ 12565540 ]
          Chris Nauroth made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Suresh Srinivas added a comment -

          +1 for the trunk patch. I will follow HADOOP-9207 to ensure the comments I made earlier is handled.

          Show
          Suresh Srinivas added a comment - +1 for the trunk patch. I will follow HADOOP-9207 to ensure the comments I made earlier is handled.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12565540/HADOOP-8924.7.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2072//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2072//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12565540/HADOOP-8924.7.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common hadoop-maven-plugins hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2072//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2072//console This message is automatically generated.
          Chris Nauroth made changes -
          Summary Hadoop Common creating package-info.java must not depend on sh, at least for Windows Add maven plugin alternative to shell script to save package-info.java
          Hide
          Hudson added a comment -

          Integrated in Hadoop-trunk-Commit #3262 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3262/)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372)

          Result = FAILURE
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435372
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-trunk-Commit #3262 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3262/ ) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372) Result = FAILURE suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435372 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-trunk-Commit #3263 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3263/)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380)
          HADOOP-8924. Revert r1435372 that missed some files (Revision 1435379)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-maven-plugins
          • /hadoop/common/trunk/hadoop-maven-plugins/pom.xml
          • /hadoop/common/trunk/hadoop-maven-plugins/src
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-trunk-Commit #3263 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3263/ ) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380) HADOOP-8924 . Revert r1435372 that missed some files (Revision 1435379) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-maven-plugins /hadoop/common/trunk/hadoop-maven-plugins/pom.xml /hadoop/common/trunk/hadoop-maven-plugins/src /hadoop/common/trunk/hadoop-maven-plugins/src/main /hadoop/common/trunk/hadoop-maven-plugins/src/main/java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435379 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Yarn-trunk #101 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/101/)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380)
          HADOOP-8924. Revert r1435372 that missed some files (Revision 1435379)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-maven-plugins
          • /hadoop/common/trunk/hadoop-maven-plugins/pom.xml
          • /hadoop/common/trunk/hadoop-maven-plugins/src
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Yarn-trunk #101 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/101/ ) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380) HADOOP-8924 . Revert r1435372 that missed some files (Revision 1435379) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-maven-plugins /hadoop/common/trunk/hadoop-maven-plugins/pom.xml /hadoop/common/trunk/hadoop-maven-plugins/src /hadoop/common/trunk/hadoop-maven-plugins/src/main /hadoop/common/trunk/hadoop-maven-plugins/src/main/java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435379 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435372 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #1290 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1290/)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380)
          HADOOP-8924. Revert r1435372 that missed some files (Revision 1435379)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372)

          Result = FAILURE
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-maven-plugins
          • /hadoop/common/trunk/hadoop-maven-plugins/pom.xml
          • /hadoop/common/trunk/hadoop-maven-plugins/src
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1290 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1290/ ) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380) HADOOP-8924 . Revert r1435372 that missed some files (Revision 1435379) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372) Result = FAILURE suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-maven-plugins /hadoop/common/trunk/hadoop-maven-plugins/pom.xml /hadoop/common/trunk/hadoop-maven-plugins/src /hadoop/common/trunk/hadoop-maven-plugins/src/main /hadoop/common/trunk/hadoop-maven-plugins/src/main/java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435379 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435372 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #1318 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1318/)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380)
          HADOOP-8924. Revert r1435372 that missed some files (Revision 1435379)
          HADOOP-8924. Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-maven-plugins
          • /hadoop/common/trunk/hadoop-maven-plugins/pom.xml
          • /hadoop/common/trunk/hadoop-maven-plugins/src
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo
          • /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties
          • /hadoop/common/trunk/hadoop-project/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties
          • /hadoop/common/trunk/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1318 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1318/ ) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435380) HADOOP-8924 . Revert r1435372 that missed some files (Revision 1435379) HADOOP-8924 . Add maven plugin alternative to shell script to save package-info.java. Contributed by Alejandro Abdelnur and Chris Nauroth. (Revision 1435372) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435380 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-maven-plugins /hadoop/common/trunk/hadoop-maven-plugins/pom.xml /hadoop/common/trunk/hadoop-maven-plugins/src /hadoop/common/trunk/hadoop-maven-plugins/src/main /hadoop/common/trunk/hadoop-maven-plugins/src/main/java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/FileSetUtils.java /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo /hadoop/common/trunk/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/versioninfo/VersionInfoMojo.java /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435379 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1435372 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/dev-support/saveVersion.sh /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/HadoopVersionAnnotation.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/VersionInfo.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/common-version-info.properties /hadoop/common/trunk/hadoop-project/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/scripts/saveVersion.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/YarnVersionInfo.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-version-info.properties /hadoop/common/trunk/pom.xml
          Hide
          Suresh Srinivas added a comment -

          I committed the patch to trunk and branch-trunk-win. Thank you Chris and Alejandro.

          BTW I could not get the merge to build in branch-2. Chris, please upload a branch-2 patch or let me know what I am missing in the merge, if you want it to be merged to branch-2.

          Show
          Suresh Srinivas added a comment - I committed the patch to trunk and branch-trunk-win. Thank you Chris and Alejandro. BTW I could not get the merge to build in branch-2. Chris, please upload a branch-2 patch or let me know what I am missing in the merge, if you want it to be merged to branch-2.
          Suresh Srinivas made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Assignee Alejandro Abdelnur [ tucu00 ] Chris Nauroth [ cnauroth ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-trunk-Commit #3264 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3264/)
          HADOOP-8924. Add CHANGES.txt description missed in commit r1435380. (Revision 1436181)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-trunk-Commit #3264 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3264/ ) HADOOP-8924 . Add CHANGES.txt description missed in commit r1435380. (Revision 1436181) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Chris Nauroth added a comment -

          Thank you, Suresh. I am attaching a branch-2 patch. The differences compared to the trunk patch are:

          1. hadoop-common-project/hadoop-common/pom.xml had diverged on branch-2, so I needed to do a (trivial) manual resolution.
          2. hadoop-maven-plugins/pom.xml references 2.0.3-SNAPSHOT as its parent version instead of 3.0.0-SNAPSHOT
          Show
          Chris Nauroth added a comment - Thank you, Suresh. I am attaching a branch-2 patch. The differences compared to the trunk patch are: hadoop-common-project/hadoop-common/pom.xml had diverged on branch-2, so I needed to do a (trivial) manual resolution. hadoop-maven-plugins/pom.xml references 2.0.3-SNAPSHOT as its parent version instead of 3.0.0-SNAPSHOT
          Chris Nauroth made changes -
          Attachment HADOOP-8924-branch-2.7.patch [ 12565730 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Yarn-trunk #103 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/103/)
          HADOOP-8924. Add CHANGES.txt description missed in commit r1435380. (Revision 1436181)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Yarn-trunk #103 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/103/ ) HADOOP-8924 . Add CHANGES.txt description missed in commit r1435380. (Revision 1436181) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #1292 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1292/)
          HADOOP-8924. Add CHANGES.txt description missed in commit r1435380. (Revision 1436181)

          Result = FAILURE
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1292 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1292/ ) HADOOP-8924 . Add CHANGES.txt description missed in commit r1435380. (Revision 1436181) Result = FAILURE suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #1320 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1320/)
          HADOOP-8924. Add CHANGES.txt description missed in commit r1435380. (Revision 1436181)

          Result = SUCCESS
          suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181
          Files :

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1320 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1320/ ) HADOOP-8924 . Add CHANGES.txt description missed in commit r1435380. (Revision 1436181) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1436181 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Colin Patrick McCabe added a comment -

          It seems like this broke mvn eclipse:eclipse.

          I get this output:

          [INFO] ------------------------------------------------------------------------
          [INFO] BUILD FAILURE
          [INFO] ------------------------------------------------------------------------
          [INFO] Total time: 6.478s
          [INFO] Finished at: Mon Jan 21 18:30:18 PST 2013
          [INFO] Final Memory: 33M/661M
          [INFO] ------------------------------------------------------------------------
          [ERROR] Plugin org.apache.hadoop:hadoop-maven-plugins:3.0.0-SNAPSHOT or one of its dependencies could not be        resolved: Could not find artifact org.apache.hadoop:hadoop-maven-plugins:jar:3.0.0-SNAPSHOT -> [Help 1]
          org.apache.maven.plugin.PluginResolutionException: Plugin org.apache.hadoop:hadoop-maven-plugins:3.0.0-SNAPSHOT or  one of its dependencies could not be resolved: Could not find artifact org.apache.hadoop:hadoop-maven-plugins:jar:3.0.0-SNAPSHOT
              at org.apache.maven.plugin.internal.DefaultPluginDependenciesResolver.resolve(DefaultPluginDependenciesResolver.java:140)
              at org.apache.maven.plugin.internal.DefaultMavenPluginManager.getPluginDescriptor(DefaultMavenPluginManager.    java:142)
              at org.apache.maven.plugin.internal.DefaultMavenPluginManager.getMojoDescriptor(DefaultMavenPluginManager.java: 261)
              at org.apache.maven.plugin.DefaultBuildPluginManager.getMojoDescriptor(DefaultBuildPluginManager.java:185)
              at org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculator.                                 calculateLifecycleMappings(DefaultLifecycleExecutionPlanCalculator.java:280)
              at org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculator.                                 calculateForkedLifecycle(DefaultLifecycleExecutionPlanCalculator.java:520)
          

          The underlying issue seems to be that generate-resources is broken, and maven-eclipse-plugin "Invokes the execution of the lifecycle phase generate-resources prior to executing itself." (As described here: http://maven.apache.org/plugins/maven-eclipse-plugin/eclipse-mojo.html )

          You can see that this command also fails with the same error: mvn generate-resources. In fact, a lot of the early phases of the default lifecycle seem to fail: validate, initialize, generate-sources. For some reason mvn site works, though, despite the fact that it should be implicitly calling those earlier build phases.

          Show
          Colin Patrick McCabe added a comment - It seems like this broke mvn eclipse:eclipse . I get this output: [INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 6.478s [INFO] Finished at: Mon Jan 21 18:30:18 PST 2013 [INFO] Final Memory: 33M/661M [INFO] ------------------------------------------------------------------------ [ERROR] Plugin org.apache.hadoop:hadoop-maven-plugins:3.0.0-SNAPSHOT or one of its dependencies could not be resolved: Could not find artifact org.apache.hadoop:hadoop-maven-plugins:jar:3.0.0-SNAPSHOT -> [Help 1] org.apache.maven.plugin.PluginResolutionException: Plugin org.apache.hadoop:hadoop-maven-plugins:3.0.0-SNAPSHOT or one of its dependencies could not be resolved: Could not find artifact org.apache.hadoop:hadoop-maven-plugins:jar:3.0.0-SNAPSHOT at org.apache.maven.plugin.internal.DefaultPluginDependenciesResolver.resolve(DefaultPluginDependenciesResolver.java:140) at org.apache.maven.plugin.internal.DefaultMavenPluginManager.getPluginDescriptor(DefaultMavenPluginManager. java:142) at org.apache.maven.plugin.internal.DefaultMavenPluginManager.getMojoDescriptor(DefaultMavenPluginManager.java: 261) at org.apache.maven.plugin.DefaultBuildPluginManager.getMojoDescriptor(DefaultBuildPluginManager.java:185) at org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculator. calculateLifecycleMappings(DefaultLifecycleExecutionPlanCalculator.java:280) at org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculator. calculateForkedLifecycle(DefaultLifecycleExecutionPlanCalculator.java:520) The underlying issue seems to be that generate-resources is broken, and maven-eclipse-plugin "Invokes the execution of the lifecycle phase generate-resources prior to executing itself." (As described here: http://maven.apache.org/plugins/maven-eclipse-plugin/eclipse-mojo.html ) You can see that this command also fails with the same error: mvn generate-resources . In fact, a lot of the early phases of the default lifecycle seem to fail: validate , initialize , generate-sources . For some reason mvn site works, though, despite the fact that it should be implicitly calling those earlier build phases.
          Hide
          Chris Nauroth added a comment -

          Hi, Colin. Sorry to hear that this patch caused trouble in your dev environment. There had been a related patch, HADOOP-9202, to change test-patch.sh to run mvn install before checking mvn eclipse:eclipse. My understanding is that the established process is to run mvn install first to guarantee that everything is deployed into your local Maven repository, and then run mvn eclipse:eclipse. That's based on the instructions shown here:

          http://wiki.apache.org/hadoop/EclipseEnvironment

          From this directory you just 'cd'-ed into (Which is also known as the top-level directory of a branch or a trunk checkout), perform:

          $ mvn install -DskipTests
          $ mvn eclipse:eclipse -DdownloadSources=true -DdownloadJavadocs=true

          Does running an mvn install first fix the issue for you? I just tested that this works in my environment by running:

          rm -rf ~/.m2/repository/org/apache/hadoop/hadoop-maven-plugins && mvn install -DskipTests && mvn eclipse:eclipse
          

          (The rm -rf was to force removal of any cached copy of the plugin that I might have in my local repository.)

          Show
          Chris Nauroth added a comment - Hi, Colin. Sorry to hear that this patch caused trouble in your dev environment. There had been a related patch, HADOOP-9202 , to change test-patch.sh to run mvn install before checking mvn eclipse:eclipse. My understanding is that the established process is to run mvn install first to guarantee that everything is deployed into your local Maven repository, and then run mvn eclipse:eclipse. That's based on the instructions shown here: http://wiki.apache.org/hadoop/EclipseEnvironment From this directory you just 'cd'-ed into (Which is also known as the top-level directory of a branch or a trunk checkout), perform: $ mvn install -DskipTests $ mvn eclipse:eclipse -DdownloadSources=true -DdownloadJavadocs=true Does running an mvn install first fix the issue for you? I just tested that this works in my environment by running: rm -rf ~/.m2/repository/org/apache/hadoop/hadoop-maven-plugins && mvn install -DskipTests && mvn eclipse:eclipse (The rm -rf was to force removal of any cached copy of the plugin that I might have in my local repository.)
          Hide
          Colin Patrick McCabe added a comment -

          Hi Chris,

          Running the install first works for me. It's not an ideal solution, but I guess we can live with it, as long as it's documented. Hopefully future versions of Maven will offer some way out of this chicken and egg problem with bundled Maven plugins.

          Now that we have a place to put Maven plugins, I guess it's time to reopen HADOOP-8887.

          By the way, thanks for doing this work.

          Show
          Colin Patrick McCabe added a comment - Hi Chris, Running the install first works for me. It's not an ideal solution, but I guess we can live with it, as long as it's documented. Hopefully future versions of Maven will offer some way out of this chicken and egg problem with bundled Maven plugins. Now that we have a place to put Maven plugins, I guess it's time to reopen HADOOP-8887 . By the way, thanks for doing this work.
          Chris Nauroth made changes -
          Link This issue breaks HADOOP-9245 [ HADOOP-9245 ]
          Suresh Srinivas made changes -
          Affects Version/s 3.0.0 [ 12320357 ]
          Chris Nauroth made changes -
          Link This issue breaks HADOOP-9249 [ HADOOP-9249 ]
          Chris Nauroth made changes -
          Link This issue breaks HADOOP-9260 [ HADOOP-9260 ]
          Eli Collins made changes -
          Link This issue is related too HADOOP-9251 [ HADOOP-9251 ]
          Chris Nauroth made changes -
          Link This issue breaks HADOOP-9279 [ HADOOP-9279 ]
          Alejandro Abdelnur made changes -
          Fix Version/s 2.0.4-beta [ 12324030 ]
          Fix Version/s trunk-win [ 12323306 ]
          Dennis Y made changes -
          Link This issue breaks HADOOP-9315 [ HADOOP-9315 ]
          Gavin made changes -
          Link This issue blocks HADOOP-9117 [ HADOOP-9117 ]
          Gavin made changes -
          Link This issue is depended upon by HADOOP-9117 [ HADOOP-9117 ]
          Gavin made changes -
          Link This issue depends on HADOOP-9116 [ HADOOP-9116 ]
          Gavin made changes -
          Link This issue depends upon HADOOP-9116 [ HADOOP-9116 ]
          Gavin made changes -
          Link This issue is related to HADOOP-9251 [ HADOOP-9251 ]
          Gavin made changes -
          Link This issue is related to HADOOP-9251 [ HADOOP-9251 ]
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Chris Nauroth
              Reporter:
              Chris Nauroth
            • Votes:
              0 Vote for this issue
              Watchers:
              14 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development