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

Jeff Zhang commented on TEZ-1233:
---------------------------------

Upload new patch to address the review comments except 1 things
bq. Do we expect any properties which may make sense only at the Vertex level, 
but not at the DAG level ?
I don't see any case of this. If this is necessary in future, I think it is 
more likely custom vertex property, not standard property in TezConfiguration.

bq. Construction of PropertyScope - is this reflection based approach expensive 
?
I think the only way to access annotation is through reflection. Are you 
suggesting the way that TezRuntimeConfiguration used ( not use annotation ) ? 

> Allow configuration of framework parameters per vertex
> ------------------------------------------------------
>
>                 Key: TEZ-1233
>                 URL: https://issues.apache.org/jira/browse/TEZ-1233
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Siddharth Seth
>            Assignee: Jeff Zhang
>         Attachments: TEZ-1233-1.patch
>
>
> Currently, configuration properties specific in AMConfiguration are used to 
> configure tasks - e.g. memory configs, process tree, etc. These should be 
> configurable at a Vertex level.
> Also, tasks end up reading these configs from the dist cache each time - 
> would be simpler to send over the wire as part of the TaskSpec.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to