You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-issues@hadoop.apache.org by "Iyappan Srinivasan (JIRA)" <ji...@apache.org> on 2010/07/21 13:39:51 UTC

[jira] Created: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
--------------------------------------------------------------------------------------

                 Key: MAPREDUCE-1955
                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
             Project: Hadoop Map/Reduce
          Issue Type: Test
          Components: test
            Reporter: Iyappan Srinivasan
            Assignee: Iyappan Srinivasan
         Attachments: JobInProgressAspectaj.patch

Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Balaji Rajagopalan (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891012#action_12891012 ] 

Balaji Rajagopalan commented on MAPREDUCE-1955:
-----------------------------------------------

+1

> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: JobInProgressAspectaj.patch, MAPREDUCE-1955.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Iyappan Srinivasan (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12890704#action_12890704 ] 

Iyappan Srinivasan commented on MAPREDUCE-1955:
-----------------------------------------------

Disregard my trunk patch. Consider only the patch for 20.1.xxx. This change is not in trunk.

> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: JobInProgressAspectaj.patch, MAPREDUCE-1955.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Iyappan Srinivasan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Iyappan Srinivasan updated MAPREDUCE-1955:
------------------------------------------

    Attachment: 1955-ydist-security-patch.txt

That patch became stale. Attachign a new patch with exactly the same changes.

> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: 1955-ydist-security-patch.txt, JobInProgressAspectaj.patch, MAPREDUCE-1955.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Iyappan Srinivasan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Iyappan Srinivasan updated MAPREDUCE-1955:
------------------------------------------

    Attachment: JobInProgressAspectaj.patch

patch for 20.1.xxx

> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: JobInProgressAspectaj.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Iyappan Srinivasan (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Iyappan Srinivasan updated MAPREDUCE-1955:
------------------------------------------

    Attachment: MAPREDUCE-1955.patch

patch for trunk

> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: JobInProgressAspectaj.patch, MAPREDUCE-1955.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MAPREDUCE-1955) Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.

Posted by "Konstantin Boudnik (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/MAPREDUCE-1955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891732#action_12891732 ] 

Konstantin Boudnik commented on MAPREDUCE-1955:
-----------------------------------------------

I'm looking into the MR trunk code and I see that {{protected AtomicBoolean tasksInited = new AtomicBoolean(false);}} Besides there's {{public boolean inited()}} to access it. So, I don't see how this patch makes any sense for trunk?



> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> --------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1955
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1955
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>            Reporter: Iyappan Srinivasan
>            Assignee: Iyappan Srinivasan
>         Attachments: 1955-ydist-security-patch.txt, JobInProgressAspectaj.patch, MAPREDUCE-1955.patch
>
>
> Because of changes in JobInProgress.java, JobInProgressAspect.aj also needs to change.
> A variable taskInited is changed from Boolean to boolean in JobInProgress.java. So JobInProgressAspect.aj  also needs to change too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.