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

Eric Yang edited comment on YARN-9561 at 5/22/19 8:58 PM:
----------------------------------------------------------

[~ebadger] Can we have some test cases?  Hadoop C is using K&R style brace 
placement, cJSON code is using Allman style brace placement.  It is a little 
confusing to see them in the same directory.  Would it be better to place cJSON 
files in a vendor sub-directory to ensure that we don't make modification to 
the source and easier to rebase to upstream?


was (Author: eyang):
[~ebadger] Can we have some test cases?  Hadoop C is using K&R style brace 
placement, cJSON code is using Allman style brace placement.  It is a little 
confusing to see them in the directory.  Would it be better to place cJSON 
files in a vendor sub-directory to ensure that we don't make modification to 
the source and easier to rebase to upstream?

> Add C changes for the new OCI/squashfs/runc runtime
> ---------------------------------------------------
>
>                 Key: YARN-9561
>                 URL: https://issues.apache.org/jira/browse/YARN-9561
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Eric Badger
>            Assignee: Eric Badger
>            Priority: Major
>         Attachments: YARN-9561.001.patch, YARN-9561.002.patch
>
>
> This JIRA will be used to add the C changes to the container-executor native 
> binary that are necessary for the new OCI/squashFS/runc runtime. There should 
> be no changes to existing code paths. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to