Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-8887

Use a Maven plugin to build the native code using CMake

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 2.0.3-alpha
    • 2.9.0, 3.0.0-alpha1
    • build
    • None
    • Reviewed

    Description

      Currently, we build the native code using ant-build invocations. Although this works, it has some limitations:

      • compiler warning messages are hidden, which can cause people to check in code with warnings unintentionally
      • there is no framework for running native unit tests; instead, we use ad-hoc constructs involving shell scripts
      • the antrun code is very platform specific
      • there is no way to run a specific native unit test
      • it's more or less impossible for scripts like test-patch.sh to separate a native test failing from the build itself failing (no files are created) or to enumerate which native tests failed.

      Using a native Maven plugin would overcome these limitations.

      Attachments

        1. HADOOP-8887.001.patch
          50 kB
          Colin McCabe
        2. HADOOP-8887.002.patch
          51 kB
          Colin McCabe
        3. HADOOP-8887.003.patch
          54 kB
          Colin McCabe
        4. HADOOP-8887.004.patch
          54 kB
          Colin McCabe
        5. HADOOP-8887.005.patch
          50 kB
          Colin McCabe
        6. HADOOP-8887.006.patch
          50 kB
          Colin McCabe
        7. HADOOP-8887.008.patch
          54 kB
          Colin McCabe
        8. HADOOP-8887.011.patch
          18 kB
          Colin McCabe
        9. HADOOP-8887.012.patch
          43 kB
          Colin McCabe
        10. HADOOP-8887.013.patch
          43 kB
          Colin McCabe
        11. HADOOP-8887.014.patch
          43 kB
          Colin McCabe

        Issue Links

          Activity

            People

              cmccabe Colin McCabe
              cmccabe Colin McCabe
              Votes:
              0 Vote for this issue
              Watchers:
              16 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: