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

Peter Bacsko commented on YARN-10018:
-------------------------------------

[~ebadger] you recently worked on the native code, do you have any idea why the 
compilation failed?

{noformat}
[WARNING] CMake Error at CMakeLists.txt:180 (add_executable):
[WARNING]   Cannot find source file:
[WARNING] 
[WARNING]     main/native/container-executor/test/utils/test_runc_util.cc
[WARNING] 
[WARNING]   Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm 
.hpp
[WARNING]   .hxx .in .txx
[WARNING] 
[WARNING] 
[WARNING] CMake Error at CMakeLists.txt:131 (add_library):
[WARNING]   Cannot find source file:
[WARNING] 
[WARNING]     main/native/container-executor/impl/utils/file-utils.c
[WARNING] 
[WARNING]   Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm 
.hpp
[WARNING]   .hxx .in .txx
[WARNING] 
[WARNING] 
[WARNING] CMake Error: Cannot determine link language for target "container".
[WARNING] CMake Error: CMake can not determine linker language for target: 
container
{noformat}

Looks like it has to do with this runc thing.

> container-executor: possible -1 return value of fork() is not always checked
> ----------------------------------------------------------------------------
>
>                 Key: YARN-10018
>                 URL: https://issues.apache.org/jira/browse/YARN-10018
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>            Reporter: Peter Bacsko
>            Assignee: Peter Bacsko
>            Priority: Major
>         Attachments: YARN-10018-001.patch
>
>
> There are some places in the container-executor native, where the {{fork()}} 
> call is not handled properly. This operation can fail with -1, but sometimes 
> the necessary if branch is missing to validate that it's been successful.
> Also, at one location, the return value is defined as an {{int}}, not 
> {{pid_t}}. It's better to handle this transparently and change it.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to