[ https://issues.apache.org/jira/browse/YARN-6033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095405#comment-16095405 ]
Wangda Tan commented on YARN-6033: ---------------------------------- [~vvasudev], While doing test for YARN-6223 (based on this patch). I think there's a minor change needed: In "container-executor.c", it has following statement: {code} executor_cfg = *(get_configuration_section("", &CFG))); {code} However if we have a config file doesn't has key/value under default section, such as an empty config file, or file like: {code} [named-section] k=v {code} The returned value of {{get_configuration_section("", &CFG)}} will be NULL which will cause process crash without proper error message. To fix the problem, we can: 1) Set an empty/NULL value to executor_cfg. Probably we should change executor_cfg to a pointer. 2) By default create an empty value (not null) section in struct configuration. Thoughts? > 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.003.patch, YARN-6033.004.patch, > 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org