giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pavan Kumar Athivarapu" <pava...@outlook.com>
Subject Re: Review Request 21151: Giraph-886 - Partition Balancing does not work
Date Thu, 08 May 2014 21:59:18 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21151/#review42543
-----------------------------------------------------------


I am not sure if u wanted to include me - (perhaps name mismatch)
if u want I can review this over weekend.

glancing over this seems good - your comment is valid. we assume sorted [& also that partitions
are consequtive]

- Pavan Kumar Athivarapu


On May 7, 2014, 10:33 a.m., Lukas Nalezenec wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21151/
> -----------------------------------------------------------
> 
> (Updated May 7, 2014, 10:33 a.m.)
> 
> 
> Review request for giraph, Avery Ching, Claudio Martella, Maja Kabiljo, pavan kumar kolamuri,
and Sebastian Schelter.
> 
> 
> Repository: giraph-git
> 
> 
> Description
> -------
> 
> PartitionBalancer returns PartitionOwners in list in arbitrary order but both Range and
Hash partitioners are assuming that list is sorted by partitionId.
> 
> 
> Diffs
> -----
> 
>   giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java 7ab291b 
>   giraph-core/src/main/java/org/apache/giraph/partition/HashWorkerPartitioner.java d833895

>   giraph-core/src/main/java/org/apache/giraph/partition/PartitionBalancer.java 3454d62

>   giraph-core/src/main/java/org/apache/giraph/partition/SimpleWorkerPartitioner.java
600d7a3 
> 
> Diff: https://reviews.apache.org/r/21151/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lukas Nalezenec
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message