[jira] [Commented] (GIRAPH-15) Use of Jenkins for tests and builds

2011-08-31 Thread Hyunsik Choi (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-15?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095069#comment-13095069 ] Hyunsik Choi commented on GIRAPH-15: FYI, I post the progress of this issue. Some jira

[jira] [Resolved] (GIRAPH-24) Job-level statistics reports one superstep greater than workers

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-24?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan resolved GIRAPH-24. --- Resolution: Fixed I've verified unit tests pass and committed this. Resolving. > Job-level statistic

[jira] [Commented] (GIRAPH-13) Port Giraph to YARN

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-13?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095043#comment-13095043 ] Avery Ching commented on GIRAPH-13: --- This is going to be a fun one. =) Thanks for taking

[jira] [Commented] (GIRAPH-24) Job-level statistics reports one superstep greater than workers

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-24?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095042#comment-13095042 ] Avery Ching commented on GIRAPH-24: --- I thought I noticed this at some point, but forgot a

[jira] [Assigned] (GIRAPH-13) Port Giraph to YARN

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-13?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan reassigned GIRAPH-13: - Assignee: Jakob Homan > Port Giraph to YARN > --- > > Key: GIRAPH-

[jira] [Commented] (GIRAPH-24) Job-level statistics reports one superstep greater than workers

2011-08-31 Thread Hyunsik Choi (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-24?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095033#comment-13095033 ] Hyunsik Choi commented on GIRAPH-24: +1 nice work On another side, we need to create j

[jira] [Created] (GIRAPH-24) Job-level statistics reports one superstep greater than workers

2011-08-31 Thread Jakob Homan (JIRA)
Job-level statistics reports one superstep greater than workers --- Key: GIRAPH-24 URL: https://issues.apache.org/jira/browse/GIRAPH-24 Project: Giraph Issue Type: Bug R

[jira] [Updated] (GIRAPH-24) Job-level statistics reports one superstep greater than workers

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-24?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan updated GIRAPH-24: -- Attachment: GIRAPH-24.patch Patch that only increments the counter after the zeroth superstep to the kee

[jira] [Commented] (GIRAPH-23) Giraph causes capacity scheduler to report crazy statistics

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-23?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094969#comment-13094969 ] Avery Ching commented on GIRAPH-23: --- That is weird, haven't seen this issue on 0.20.203 a

[jira] [Created] (GIRAPH-23) Giraph causes capacity scheduler to report crazy statistics

2011-08-31 Thread Jakob Homan (JIRA)
Giraph causes capacity scheduler to report crazy statistics --- Key: GIRAPH-23 URL: https://issues.apache.org/jira/browse/GIRAPH-23 Project: Giraph Issue Type: Bug Environment:

[jira] [Commented] (GIRAPH-22) Sort out examples from unit test helpers in examples package

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-22?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094901#comment-13094901 ] Avery Ching commented on GIRAPH-22: --- Good idea. I think a few of them are full programs

[jira] [Created] (GIRAPH-22) Sort out examples from unit test helpers in examples package

2011-08-31 Thread Jakob Homan (JIRA)
Sort out examples from unit test helpers in examples package Key: GIRAPH-22 URL: https://issues.apache.org/jira/browse/GIRAPH-22 Project: Giraph Issue Type: Improvement

[jira] [Commented] (GIRAPH-19) Create a CHANGES.txt file

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-19?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094840#comment-13094840 ] Avery Ching commented on GIRAPH-19: --- Should we add dates too? I don't have a preference

[jira] [Resolved] (GIRAPH-19) Create a CHANGES.txt file

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-19?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan resolved GIRAPH-19. --- Resolution: Fixed This file was created as part of GIRAPH-16. Resolving. I would note that entries

[jira] [Updated] (GIRAPH-18) Refactor BspServiceWorker::loadVertices()

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-18?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan updated GIRAPH-18: -- Attachment: GIRAPH-18c.patch Updated the comment. Attaching just for paperwork's sake and ensure this

[jira] [Commented] (GIRAPH-18) Refactor BspServiceWorker::loadVertices()

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-18?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094793#comment-13094793 ] Avery Ching commented on GIRAPH-18: --- +1 Looks better, thanks. Hey, could you add a javad

[jira] [Resolved] (GIRAPH-16) Add Apache RAT to maven build

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan resolved GIRAPH-16. --- Resolution: Fixed Owen committed this. Resolving. > Add Apache RAT to maven build >

[jira] [Assigned] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Avery Ching reassigned GIRAPH-21: - Assignee: Avery Ching > Revise CODE_CONVENTIONS > --- > > Key:

[jira] [Commented] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094768#comment-13094768 ] Avery Ching commented on GIRAPH-21: --- I'll definitely let this continue to flesh out, but

[jira] [Commented] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Hyunsik Choi (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094757#comment-13094757 ] Hyunsik Choi commented on GIRAPH-21: I prefer 80 chars per line and 2 space indent. >

[jira] [Updated] (GIRAPH-18) Refactor BspServiceWorker::loadVertices()

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-18?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jakob Homan updated GIRAPH-18: -- Attachment: GIRAPH-18b.patch Updated patch. Removed all the two-space idents I could find, fixed point

[jira] [Commented] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Henry Saputra (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094731#comment-13094731 ] Henry Saputra commented on GIRAPH-21: - I like the 2 space indent, spaces only, and max

[jira] [Commented] (GIRAPH-19) Create a CHANGES.txt file

2011-08-31 Thread Hyunsik Choi (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-19?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094732#comment-13094732 ] Hyunsik Choi commented on GIRAPH-19: agreed +1 > Create a CHANGES.txt file > -

[jira] [Commented] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094718#comment-13094718 ] Owen O'Malley commented on GIRAPH-21: - Where I almost always use emacs and so anything

[jira] [Commented] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-21?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094714#comment-13094714 ] Jakob Homan commented on GIRAPH-21: --- I usually think of 80 chars as being 80 + 3 or 4 cha

[jira] [Created] (GIRAPH-21) Revise CODE_CONVENTIONS

2011-08-31 Thread Avery Ching (JIRA)
Revise CODE_CONVENTIONS --- Key: GIRAPH-21 URL: https://issues.apache.org/jira/browse/GIRAPH-21 Project: Giraph Issue Type: Improvement Reporter: Avery Ching Priority: Minor Currently there is a

Re: dev process

2011-08-31 Thread Hyunsik Choi
+1 Review than commit process -- Hyunsik Choi On Thu, Sep 1, 2011 at 1:13 AM, Avery Ching wrote: > I agree that RTC is better for us, although in case of build breakage, I'm > okay with CTR in that case. > > Avery > > On Aug 31, 2011, at 9:09 AM, Jakob Homan wrote: > >> RTC is definitely the w

[jira] [Resolved] (GIRAPH-14) Support for the Facebook Hadoop branch

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-14?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Avery Ching resolved GIRAPH-14. --- Resolution: Fixed Committed, with changelog addition. > Support for the Facebook Hadoop branch >

[jira] [Commented] (GIRAPH-18) Refactor BspServiceWorker::loadVertices()

2011-08-31 Thread Avery Ching (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-18?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094686#comment-13094686 ] Avery Ching commented on GIRAPH-18: --- +1 Nice work refactoring, makes the code more reada

Re: dev process

2011-08-31 Thread Avery Ching
I agree that RTC is better for us, although in case of build breakage, I'm okay with CTR in that case. Avery On Aug 31, 2011, at 9:09 AM, Jakob Homan wrote: > RTC is definitely the way to go (he said with a weary sigh). > > On Wed, Aug 31, 2011 at 6:32 AM, Owen O'Malley wrote: >> All, >> It

[jira] [Commented] (GIRAPH-16) Add Apache RAT to maven build

2011-08-31 Thread Jakob Homan (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094646#comment-13094646 ] Jakob Homan commented on GIRAPH-16: --- +1 > Add Apache RAT to maven build > --

Re: dev process

2011-08-31 Thread Jakob Homan
RTC is definitely the way to go (he said with a weary sigh). On Wed, Aug 31, 2011 at 6:32 AM, Owen O'Malley wrote: > All, >   It seems that we've implicitly picked review then commit (RTC) instead of > commit then review (CTR). Apache projects allow either approach and I'm fine > with either. W

[jira] [Commented] (GIRAPH-15) Use of Jenkins for tests and builds

2011-08-31 Thread Hyunsik Choi (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-15?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094584#comment-13094584 ] Hyunsik Choi commented on GIRAPH-15: Thank you for informing :) I'll ask Chris for that

[jira] [Commented] (GIRAPH-15) Use of Jenkins for tests and builds

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-15?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13094569#comment-13094569 ] Owen O'Malley commented on GIRAPH-15: - I'm already in the hudson-jobadmin group, but it

[jira] [Updated] (GIRAPH-4) New project logo

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-4?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Owen O'Malley updated GIRAPH-4: --- Component/s: site > New project logo > > > Key: GIRAPH-4 >

[jira] [Updated] (GIRAPH-16) Add Apache RAT to maven build

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Owen O'Malley updated GIRAPH-16: Component/s: build > Add Apache RAT to maven build > - > >

[jira] [Updated] (GIRAPH-20) Move temporary test files from the project directory

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-20?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Owen O'Malley updated GIRAPH-20: Component/s: test > Move temporary test files from the project directory > -

[jira] [Updated] (GIRAPH-19) Create a CHANGES.txt file

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-19?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Owen O'Malley updated GIRAPH-19: Component/s: documentation > Create a CHANGES.txt file > - > >

[jira] [Created] (GIRAPH-20) Move temporary test files from the project directory

2011-08-31 Thread Owen O'Malley (JIRA)
Move temporary test files from the project directory Key: GIRAPH-20 URL: https://issues.apache.org/jira/browse/GIRAPH-20 Project: Giraph Issue Type: Improvement Reporter: Owen

dev process

2011-08-31 Thread Owen O'Malley
All, It seems that we've implicitly picked review then commit (RTC) instead of commit then review (CTR). Apache projects allow either approach and I'm fine with either. We should just state what we are doing. I'd also like to propose that we keep a CHANGES.txt file that includes who contri

[jira] [Created] (GIRAPH-19) Create a CHANGES.txt file

2011-08-31 Thread Owen O'Malley (JIRA)
Create a CHANGES.txt file - Key: GIRAPH-19 URL: https://issues.apache.org/jira/browse/GIRAPH-19 Project: Giraph Issue Type: Improvement Reporter: Owen O'Malley Assignee: Owen O'Malley It is h

[jira] [Updated] (GIRAPH-16) Add Apache RAT to maven build

2011-08-31 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Owen O'Malley updated GIRAPH-16: Attachment: rat.patch This adds rat, which analyzes the project for files that are missing the apach