Hadoop Common
  1. Hadoop Common
  2. HADOOP-7869

HADOOP_HOME warning happens all of the time

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.0.0
    • Component/s: scripts
    • Labels:
      None
    • Target Version/s:

      Description

      With HADOOP-7816, the check for HADOOP_HOME has moved after it is set by hadoop-config so that it always happens unless HADOOP_HOME_WARN_SUPPRESS is set in hadoop-env or the environment.

      1. h-7869.patch
        1 kB
        Owen O'Malley
      2. h-7869.patch
        2 kB
        Owen O'Malley

        Activity

        Hide
        Owen O'Malley added a comment -

        This patch moves the check into hadoop-config and suppresses further warnings.

        Show
        Owen O'Malley added a comment - This patch moves the check into hadoop-config and suppresses further warnings.
        Hide
        Owen O'Malley added a comment -

        I messed up the fix from HADOOP-7816. This pulls the hadoop-env into hadoop-config.

        Show
        Owen O'Malley added a comment - I messed up the fix from HADOOP-7816 . This pulls the hadoop-env into hadoop-config.
        Hide
        Arpit Gupta added a comment -

        applied the patch and built hadoop and ran the following tests

        1. No HADOOP_HOME exported -> no warning
        2. HADOOP_HOME exported -> warning
        3. HADOOP_HOME and HADOOP_HOME_WARN_SUPPRESS set -> no warning
        4. HADOOP_HOME set and HADOOP_HOME_WARN_S +UPRESS set in hadoop-env.sh -> no warning.

        So it is working as expected. +1

        Show
        Arpit Gupta added a comment - applied the patch and built hadoop and ran the following tests 1. No HADOOP_HOME exported -> no warning 2. HADOOP_HOME exported -> warning 3. HADOOP_HOME and HADOOP_HOME_WARN_SUPPRESS set -> no warning 4. HADOOP_HOME set and HADOOP_HOME_WARN_S +UPRESS set in hadoop-env.sh -> no warning. So it is working as expected. +1
        Hide
        Matt Foley added a comment -

        Committed to 1.0.0 and branch-1. Thanks, Owen!

        Per HADOOP-7816, not needed on trunk.

        Show
        Matt Foley added a comment - Committed to 1.0.0 and branch-1. Thanks, Owen! Per HADOOP-7816 , not needed on trunk.
        Hide
        Matt Foley added a comment -

        Closed upon release of version 1.0.0.

        Show
        Matt Foley added a comment - Closed upon release of version 1.0.0.

          People

          • Assignee:
            Owen O'Malley
            Reporter:
            Owen O'Malley
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development