hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eli Collins (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-6605) Add JAVA_HOME detection to hadoop-config
Date Wed, 18 May 2011 18:14:47 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-6605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13035537#comment-13035537
] 

Eli Collins commented on HADOOP-6605:
-------------------------------------

Allen raised some good questions earlier...

bq. Why doesn't this try the java that is the path? 

I assume you mean that if java is in the path it could infer JAVA_HOME by looking at the parent
of the containing directory. That's reasonable, think it would make sense to do this in addition
to looking in standard locations (since we do care about the java we're looking for). If you
feel strongly I can incorporate that into this change.

bq. Why doesn't this verify that $JAVA_HOME/bin/java actually works?

I think it's OK to assume that the java installed in java home works.

bq. Shouldn't it verify that it found JDK6 or better?

That would be a good improvement, it also needs to check for Sun-compatible Java since Hadoop
currently depends on Sun-specific classes. Let's handle this in a separate jira.

bq. Why is the error message RH-based specific? Heck, why have the "please download" message
at all?

You're right, it shouldn't be. The latest patch is just a sligh modification to the current
error message.

> Add JAVA_HOME detection to hadoop-config
> ----------------------------------------
>
>                 Key: HADOOP-6605
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6605
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Chad Metcalf
>            Assignee: Eli Collins
>            Priority: Minor
>             Fix For: 0.22.0
>
>         Attachments: HADOOP-6605.patch, hadoop-6605-1.patch
>
>
> The commands that source hadoop-config.sh currently bail with an error if JAVA_HOME is
not set. Let's detect JAVA_HOME (from a list of locations on various OS types) if JAVA_HOME
is not already set by hadoop-env.sh or the environment. This way users don't have to manually
configure it.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message