[jira] [Commented] (YARN-6033) Add support for sections in container-executor configuration file

["Wangda Tan (JIRA)" <ji...@apache.org>]

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

Wangda Tan commented on YARN-6033:
----------------------------------

Thanks [~vvasudev] for the patch. I just took a look at the patch, in general it is in good shape, in addition to the configuration section itself, it also brings gtest framework, which will be much easier to add tests in the future.

Some comments:
1) It's better to move {{struct section executor_cfg = {.size=0, .sectiondetails=NULL};}} and {{struct configuration CFG = {.size=0, .sections=NULL};}} from container-executor.c to configurations.c. And add getter/setter method to configuration.h. I think we should not couple life cycle of configuration and container-executor since we could add other modules beyond container-executor in the new design.
2) some rename suggestions: 
- sectionentry: is it better to call {{kv_pair}}? 
- sectiondetails: if you agree with above, how about rename it to {{kv_pairs}}?

 [~sunilg].

> Add support for sections in container-executor configuration file
> -----------------------------------------------------------------
>
>                 Key: YARN-6033
>                 URL: https://issues.apache.org/jira/browse/YARN-6033
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>            Reporter: Varun Vasudev
>            Assignee: Varun Vasudev
>         Attachments: YARN-6033-YARN-5673.001.patch, YARN-6033-YARN-5673.002.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org