You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@impala.apache.org by "Joe McDonnell (Jira)" <ji...@apache.org> on 2022/05/22 03:15:00 UTC

[jira] [Resolved] (IMPALA-10699) Apply patch to libev to support compiling with C++17

     [ https://issues.apache.org/jira/browse/IMPALA-10699?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe McDonnell resolved IMPALA-10699.
------------------------------------
    Fix Version/s: Impala 4.2.0
       Resolution: Fixed

> Apply patch to libev to support compiling with C++17
> ----------------------------------------------------
>
>                 Key: IMPALA-10699
>                 URL: https://issues.apache.org/jira/browse/IMPALA-10699
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Infrastructure
>    Affects Versions: Impala 4.0.0
>            Reporter: Joe McDonnell
>            Assignee: Joe McDonnell
>            Priority: Major
>             Fix For: Impala 4.2.0
>
>
> When trying to compile with -std=c++17, multiple files fail to compile due to c++17 incompatible code in libev:
> {noformat}
> In file included from /home/joe/view2/Impala/be/src/kudu/rpc/connection.h:30:0,
>                  from /home/joe/view2/Impala/be/src/kudu/rpc/messenger.h:32,
>                  from /home/joe/view2/Impala/be/src/rpc/rpc-mgr.h:22,
>                  from /home/joe/view2/Impala/be/src/service/data-stream-service.cc:27:
> /opt/Impala-Toolchain/toolchain-packages-gcc7.5.0/libev-4.20/include/ev++.h:355:46: error: ISO C++1z does not allow dynamic exception specifications
>      dynamic_loop (unsigned int flags = AUTO) throw (bad_loop)
>                                               ^~~~~
> {noformat}
> Kudu already applies a patch to libev's ev++.h to fix this, so we should apply the same patch to libev in native-toolchain. 
> [https://github.com/apache/kudu/blob/master/thirdparty/patches/libev-c17.patch]



--
This message was sent by Atlassian Jira
(v8.20.7#820007)