Return-Path: X-Original-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id F0043C243 for ; Tue, 29 May 2012 00:22:23 +0000 (UTC) Received: (qmail 21770 invoked by uid 500); 29 May 2012 00:22:23 -0000 Delivered-To: apmail-hadoop-common-issues-archive@hadoop.apache.org Received: (qmail 21717 invoked by uid 500); 29 May 2012 00:22:23 -0000 Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: common-issues@hadoop.apache.org Delivered-To: mailing list common-issues@hadoop.apache.org Received: (qmail 21705 invoked by uid 99); 29 May 2012 00:22:23 -0000 Received: from issues-vm.apache.org (HELO issues-vm) (140.211.11.160) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 May 2012 00:22:23 +0000 Received: from isssues-vm.apache.org (localhost [127.0.0.1]) by issues-vm (Postfix) with ESMTP id 58CA2140B94 for ; Tue, 29 May 2012 00:22:23 +0000 (UTC) Date: Tue, 29 May 2012 00:22:23 +0000 (UTC) From: "Ivan Mitic (JIRA)" To: common-issues@hadoop.apache.org Message-ID: <874925870.9528.1338250943365.JavaMail.jiratomcat@issues-vm> In-Reply-To: <2032819837.9416.1337718461018.JavaMail.jiratomcat@issues-vm> Subject: [jira] [Commented] (HADOOP-8424) Web UI broken on Windows because classpath not setup correctly MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HADOOP-8424?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13284588#comment-13284588 ] Ivan Mitic commented on HADOOP-8424: ------------------------------------ Looks good overall. Three minor comments: 1. It seems that you misplaced the comment "for developers, add Hadoop classes to CLASSPATH", it should be left on its original location 2. Can you please remove the "if exist %HADOOP_CORE_HOME%\..." before "for" loops as it is not needed 3. Can you also please break the two if's and place: {code} for %%i in (%HADOOP_CORE_HOME%\build\*.jar) do ( set CLASSPATH=!CLASSPATH!;%%i ) {code} under "for releases, add core hadoop jar & webapps to CLASSPATH", and place: {code} for %%i in (%HADOOP_CORE_HOME%\build\ivy\lib\Hadoop\common\*.jar) do ( set CLASSPATH=!CLASSPATH!;%%i ) {code} under: "add libs to CLASSPATH" right after {code}for %%i in (%HADOOP_CORE_HOME%\lib\*.jar) do ( set CLASSPATH=!CLASSPATH!;%%i ){code} This way, we will be more consistent with the script layout from "bin\hadoop" used for non-Windows platforms. Separately, what do you think about having a tracking Jira on making .sh and .cmd scripts fully consistent, as they aren't at the moment? > Web UI broken on Windows because classpath not setup correctly > -------------------------------------------------------------- > > Key: HADOOP-8424 > URL: https://issues.apache.org/jira/browse/HADOOP-8424 > Project: Hadoop Common > Issue Type: Bug > Affects Versions: 1.0.0 > Reporter: Bikas Saha > Assignee: Bikas Saha > Fix For: 1.1.0 > > Attachments: HADOOP-8424.branch-1-win.patch > > > The classpath is setup to include the hadoop jars before the build webapps directory and that upsets jetty when it is trying to resolve the webapp classes. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira