[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-14 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171736#comment-14171736 ] Bikas Saha commented on TEZ-1666: - Not sure what is expected to be null. TezUserPayload obje

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-14 Thread Jeff Zhang (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171740#comment-14171740 ] Jeff Zhang commented on TEZ-1666: - It looks like Context.getUserPayload won't be null, found

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-14 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171761#comment-14171761 ] Siddharth Seth commented on TEZ-1666: - Ideally, UserPayload should be null - because tha

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172713#comment-14172713 ] Bikas Saha commented on TEZ-1666: - bq. If ProcessorDescriptor.setUserPayload were accepting

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172817#comment-14172817 ] Siddharth Seth commented on TEZ-1666: - bq. If ProcDesc.setUserPayload() not called by th

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172826#comment-14172826 ] Bikas Saha commented on TEZ-1666: - The disadvantage is that if there is any additional data

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172948#comment-14172948 ] Siddharth Seth commented on TEZ-1666: - bq. E.g. all the user overriden client side confs

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Hitesh Shah (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172963#comment-14172963 ] Hitesh Shah commented on TEZ-1666: -- [~bikassaha] Going back to your comment: bq. If Proc

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173005#comment-14173005 ] Bikas Saha commented on TEZ-1666: - Folks, this is just a question of documentation of semant

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-15 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173170#comment-14173170 ] Siddharth Seth commented on TEZ-1666: - bq. Context.getUserPayload() will return a UserPa

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-22 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14180202#comment-14180202 ] Bikas Saha commented on TEZ-1666: - [~hitesh][~sseth] From what I see, this can be simply fix

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-28 Thread Hitesh Shah (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14187642#comment-14187642 ] Hitesh Shah commented on TEZ-1666: -- Looks ok for the most part. {code} public T setUse

[jira] [Commented] (TEZ-1666) UserPayload should be null if the payload is not specified

2014-10-29 Thread Hitesh Shah (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14188981#comment-14188981 ] Hitesh Shah commented on TEZ-1666: -- +1. With the version check in place, 0.5.1 clients cann