Return-Path: X-Original-To: apmail-giraph-dev-archive@www.apache.org Delivered-To: apmail-giraph-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A599F9813 for ; Sat, 4 Aug 2012 00:49:02 +0000 (UTC) Received: (qmail 53706 invoked by uid 500); 4 Aug 2012 00:49:02 -0000 Delivered-To: apmail-giraph-dev-archive@giraph.apache.org Received: (qmail 53669 invoked by uid 500); 4 Aug 2012 00:49:02 -0000 Mailing-List: contact dev-help@giraph.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@giraph.apache.org Delivered-To: mailing list dev@giraph.apache.org Received: (qmail 53660 invoked by uid 500); 4 Aug 2012 00:49:02 -0000 Delivered-To: apmail-incubator-giraph-dev@incubator.apache.org Received: (qmail 53656 invoked by uid 99); 4 Aug 2012 00:49:02 -0000 Received: from issues-vm.apache.org (HELO issues-vm) (140.211.11.160) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 04 Aug 2012 00:49:02 +0000 Received: from isssues-vm.apache.org (localhost [127.0.0.1]) by issues-vm (Postfix) with ESMTP id 66F84140541 for ; Sat, 4 Aug 2012 00:49:02 +0000 (UTC) Date: Sat, 4 Aug 2012 00:49:02 +0000 (UTC) From: "Eli Reisman (JIRA)" To: giraph-dev@incubator.apache.org Message-ID: <148499455.12769.1344041342423.JavaMail.jiratomcat@issues-vm> In-Reply-To: <1454541360.59579.1342464395131.JavaMail.jiratomcat@issues-vm> Subject: [jira] [Commented] (GIRAPH-258) Check type compatibility before submitting job 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/GIRAPH-258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13428510#comment-13428510 ] Eli Reisman commented on GIRAPH-258: ------------------------------------ All of the code is copied over from the GiraphMapper impl where the generics were checked after the job was already running, test code didn't change, just added the validator to it. I will remove check style comments. Those were the pre-existing tests as well, and they work so I left them alone. I can change the unchecked exceptions to fail() calls in the tests (that never made any sense but I figured it was a separate issue) further (very reasonable) fixes to the existing code I stole/moved might deserve its own JIRA. Fixing now... > Check type compatibility before submitting job > ---------------------------------------------- > > Key: GIRAPH-258 > URL: https://issues.apache.org/jira/browse/GIRAPH-258 > Project: Giraph > Issue Type: Bug > Affects Versions: 0.2.0 > Reporter: Jakob Homan > Assignee: Eli Reisman > Fix For: 0.2.0 > > Attachments: GIRAPH-258-1.patch > > > Currently if one submits a job with a vertex and inputformat types that are not compatible the job only fails during runtime. Since we know the types at submission time, we can check this before submitting and fail then. -- 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