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

Hudson commented on GIRAPH-500:
-------------------------------

Integrated in Giraph-trunk-Commit #670 (See 
[https://builds.apache.org/job/Giraph-trunk-Commit/670/])
    GIRAPH-500: Refactor job launch code out of graph package and into job 
package (ereisman) (Revision 0c77d52f3c7eee138f36eeb831b4afab45216b2f)

     Result = SUCCESS
ereisman : 
http://git-wip-us.apache.org/repos/asf?p=giraph.git&a=commit&h=0c77d52f3c7eee138f36eeb831b4afab45216b2f
Files : 
* giraph-core/src/main/java/org/apache/giraph/graph/DefaultJobObserver.java
* 
giraph-core/src/test/java/org/apache/giraph/aggregators/TestAggregatorsHandling.java
* 
giraph-core/src/main/java/org/apache/giraph/benchmark/AggregatorsBenchmark.java
* giraph-core/src/main/java/org/apache/giraph/utils/InternalVertexRunner.java
* 
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/TestHBaseRootMarkerVertextFormat.java
* 
giraph-core/src/main/java/org/apache/giraph/examples/SimpleCheckpointVertex.java
* giraph-core/src/test/java/org/apache/giraph/TestGraphPartitioner.java
* giraph-core/src/test/java/org/apache/giraph/TestManualCheckpoint.java
* giraph-core/src/test/java/org/apache/giraph/io/TestJsonBase64Format.java
* giraph-core/src/main/java/org/apache/giraph/benchmark/PageRankBenchmark.java
* giraph-core/src/main/java/org/apache/giraph/job/package-info.java
* giraph-core/src/main/java/org/apache/giraph/graph/GiraphTypeValidator.java
* giraph-core/src/test/java/org/apache/giraph/TestAutoCheckpoint.java
* giraph-core/src/main/java/org/apache/giraph/job/DefaultJobObserver.java
* giraph-core/src/main/java/org/apache/giraph/graph/GiraphJob.java
* giraph-core/src/test/java/org/apache/giraph/BspCase.java
* giraph-core/src/test/java/org/apache/giraph/TestNotEnoughMapTasks.java
* giraph-core/src/main/java/org/apache/giraph/graph/GiraphJobObserver.java
* giraph-core/src/test/java/org/apache/giraph/examples/TestPageRank.java
* 
giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/TestAccumuloVertexFormat.java
* giraph-core/src/main/java/org/apache/giraph/job/GiraphJobObserver.java
* 
giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java
* giraph-core/src/test/java/org/apache/giraph/TestBspBasic.java
* giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java
* giraph-core/src/main/java/org/apache/giraph/GiraphRunner.java
* giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java
* CHANGELOG
* giraph-core/src/test/java/org/apache/giraph/TestMutateGraph.java
* 
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java
* 
giraph-core/src/main/java/org/apache/giraph/examples/SimpleVertexWithWorkerContext.java
* 
giraph-core/src/main/java/org/apache/giraph/benchmark/RandomMessageBenchmark.java
* 
giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HiveGiraphRunner.java
* giraph-core/src/test/java/org/apache/giraph/vertex/TestVertexTypes.java
* 
giraph-core/src/main/java/org/apache/giraph/benchmark/ShortestPathsBenchmark.java
* giraph-core/src/main/java/org/apache/giraph/job/GiraphTypeValidator.java

                
> Refactor job launch code out of graph package and into job package
> ------------------------------------------------------------------
>
>                 Key: GIRAPH-500
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-500
>             Project: Giraph
>          Issue Type: Improvement
>            Reporter: Eli Reisman
>            Assignee: Eli Reisman
>            Priority: Minor
>         Attachments: GIRAPH-500-1.patch
>
>
> There's too many classes in graph and its become a catch-all for all sorts of 
> stuff again. I'm moving the 4 classes that relate to job launch into a new 
> "org.apache.giraph.job" package. I left GiraphTaskManager in graph since it 
> is the beginning (nowadays) of the real BSP/Giraph-only activity that does 
> not relate to how we got to that point (Hadoop mappers running, YARN resource 
> request, whatever) in the job run. All the preliminary stuff can live in 
> o.a.g.job now. 
> I also left GiraphRunner on the top-level of the source tree since it is easy 
> for new folks to find there and this way we don't break anyone's existing 
> command lines. We can move it in a future JIRA, or not at all.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to