Return-Path: Delivered-To: apmail-hadoop-core-dev-archive@www.apache.org Received: (qmail 70165 invoked from network); 6 Sep 2008 23:10:40 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 6 Sep 2008 23:10:40 -0000 Received: (qmail 91018 invoked by uid 500); 6 Sep 2008 23:10:33 -0000 Delivered-To: apmail-hadoop-core-dev-archive@hadoop.apache.org Received: (qmail 90985 invoked by uid 500); 6 Sep 2008 23:10:32 -0000 Mailing-List: contact core-dev-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: core-dev@hadoop.apache.org Delivered-To: mailing list core-dev@hadoop.apache.org Received: (qmail 90974 invoked by uid 99); 6 Sep 2008 23:10:32 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 06 Sep 2008 16:10:32 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 06 Sep 2008 23:09:43 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 3BB67234C1C5 for ; Sat, 6 Sep 2008 16:09:44 -0700 (PDT) Message-ID: <160159278.1220742584243.JavaMail.jira@brutus> Date: Sat, 6 Sep 2008 16:09:44 -0700 (PDT) From: "Hadoop QA (JIRA)" To: core-dev@hadoop.apache.org Subject: [jira] Commented: (HADOOP-3963) libhdfs should never exit on its own but rather return errors to the calling application In-Reply-To: <1799744464.1218838184211.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HADOOP-3963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12628924#action_12628924 ] Hadoop QA commented on HADOOP-3963: ----------------------------------- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12389619/patch1.txt against trunk revision 692700. +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 tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3199/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3199/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3199/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3199/console This message is automatically generated. > libhdfs should never exit on its own but rather return errors to the calling application > ---------------------------------------------------------------------------------------- > > Key: HADOOP-3963 > URL: https://issues.apache.org/jira/browse/HADOOP-3963 > Project: Hadoop Core > Issue Type: Improvement > Components: libhdfs > Reporter: Pete Wyckoff > Assignee: Pete Wyckoff > Priority: Minor > Fix For: 0.19.0 > > Attachments: patch0.txt, patch1.txt > > > A a library, libhdfs would be better to return errors rather than directly exiting. This way the calling application can determine what to do on error. And it is more unix library like to handle errors this way. > Some applications like fuse-dfs need to handle the exit themselves after emitting an appropriate error message to its log. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.