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 675671040D for ; Tue, 9 Jul 2013 16:29:13 +0000 (UTC) Received: (qmail 12178 invoked by uid 500); 9 Jul 2013 16:29:13 -0000 Delivered-To: apmail-giraph-dev-archive@giraph.apache.org Received: (qmail 11960 invoked by uid 500); 9 Jul 2013 16:29:11 -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 11942 invoked by uid 99); 9 Jul 2013 16:29:10 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 09 Jul 2013 16:29:10 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 22C451CE4E5; Tue, 9 Jul 2013 16:29:00 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============2449552076712682888==" MIME-Version: 1.0 Subject: Re: Review Request 12322: GIRAPH-709: More flexible Jython script loading From: "Nitay Joffe" To: "Avery Ching" , "giraph" , "Nitay Joffe" Date: Tue, 09 Jul 2013 16:29:00 -0000 Message-ID: <20130709162900.14335.83382@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Nitay Joffe" X-ReviewGroup: giraph X-ReviewRequest-URL: https://reviews.apache.org/r/12322/ X-Sender: "Nitay Joffe" References: <20130709065426.14364.97375@reviews.apache.org> In-Reply-To: <20130709065426.14364.97375@reviews.apache.org> Reply-To: "Nitay Joffe" --===============2449552076712682888== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > On July 9, 2013, 6:54 a.m., Avery Ching wrote: > > giraph-core/src/main/java/org/apache/giraph/conf/JsonStringConfOption.java, line 89 > > > > > > Isn't this a real error? Yes you're right I'll pass the IOException up. > On July 9, 2013, 6:54 a.m., Avery Ching wrote: > > giraph-core/src/main/java/org/apache/giraph/conf/JsonStringConfOption.java, lines 135-145 > > > > > > Usually > > @Override is on the previous line. Yeah my intellij keeps generating them this way, will fix. > On July 9, 2013, 6:54 a.m., Avery Ching wrote: > > giraph-core/src/main/java/org/apache/giraph/jython/JythonLoader.java, lines 69-70 > > > > > > I don't see where these are called. Why would we want to do this? Would it be in the case of say one for computation, one for combiner, etc? Yes exactly, supporting any number of jython scripts so user could split things up / modularize. - Nitay ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12322/#review22872 ----------------------------------------------------------- On July 8, 2013, 10:36 p.m., Nitay Joffe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12322/ > ----------------------------------------------------------- > > (Updated July 8, 2013, 10:36 p.m.) > > > Review request for giraph. > > > Bugs: GIRAPH-709 > https://issues.apache.org/jira/browse/GIRAPH-709 > > > Repository: giraph-git > > > Description > ------- > > See JIRA. > > > Diffs > ----- > > giraph-core/src/main/java/org/apache/giraph/benchmark/PageRankBenchmark.java 413107dd70ba48ab7647127481ff47039fd1a758 > giraph-core/src/main/java/org/apache/giraph/conf/JsonStringConfOption.java PRE-CREATION > giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java e7af82565f38fbaafd06b4579acd4dce48e6f027 > giraph-core/src/main/java/org/apache/giraph/jython/DeployedScript.java PRE-CREATION > giraph-core/src/main/java/org/apache/giraph/jython/JythonComputationFactory.java f7331acc99909e665fae869a5fdb31d82b0a6e5c > giraph-core/src/main/java/org/apache/giraph/jython/JythonLoader.java PRE-CREATION > giraph-core/src/main/java/org/apache/giraph/jython/JythonUtils.java 77040e309e873fbc3394ae144176f10b95e85faa > giraph-core/src/main/java/org/apache/giraph/utils/ConfigurationUtils.java aba51318b052693440ccb6131ae7bb3120bdac96 > giraph-core/src/test/java/org/apache/giraph/jython/TestJython.java 58f25a67cdcf705f9cd845256ac6ebce8b4cc779 > > Diff: https://reviews.apache.org/r/12322/diff/ > > > Testing > ------- > > > Thanks, > > Nitay Joffe > > --===============2449552076712682888==--