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

jackylau updated FLINK-28583:
-----------------------------
    Description: 
flink don't shade the log4j and want to put it flink/lib using shade exclusion 
like this 
{code:java}
<excludes>
   <!-- log4j 2 is bundled separately from the flink-dist jar -->
   <exclude>org.apache.logging.log4j:*</exclude> {code}
and add this to put them to flink/lib
{code:java}
<includes>
   <include>org.apache.logging.log4j:log4j-api</include>
   <include>org.apache.logging.log4j:log4j-core</include>
   <include>org.apache.logging.log4j:log4j-slf4j-impl</include>
   <include>org.apache.logging.log4j:log4j-1.2-api</include>
</includes> {code}
i suggest to make the log4j to provided.

 

the assemble dependencySet scope just like filter 

  was:
flink don't shade the log4j and want to put it flink/lib using shade exclusion 
like this 
{code:java}
<excludes>
   <!-- log4j 2 is bundled separately from the flink-dist jar -->
   <exclude>org.apache.logging.log4j:*</exclude> {code}
and add this to put them to flink/lib
{code:java}
<includes>
   <include>org.apache.logging.log4j:log4j-api</include>
   <include>org.apache.logging.log4j:log4j-core</include>
   <include>org.apache.logging.log4j:log4j-slf4j-impl</include>
   <include>org.apache.logging.log4j:log4j-1.2-api</include>
</includes> {code}
i suggest to make the log4j to provided.


> make flink dist log4j dependency simple and clear
> -------------------------------------------------
>
>                 Key: FLINK-28583
>                 URL: https://issues.apache.org/jira/browse/FLINK-28583
>             Project: Flink
>          Issue Type: Improvement
>          Components: Deployment / Scripts
>    Affects Versions: 1.16.0
>            Reporter: jackylau
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.16.0
>
>
> flink don't shade the log4j and want to put it flink/lib using shade 
> exclusion like this 
> {code:java}
> <excludes>
>    <!-- log4j 2 is bundled separately from the flink-dist jar -->
>    <exclude>org.apache.logging.log4j:*</exclude> {code}
> and add this to put them to flink/lib
> {code:java}
> <includes>
>    <include>org.apache.logging.log4j:log4j-api</include>
>    <include>org.apache.logging.log4j:log4j-core</include>
>    <include>org.apache.logging.log4j:log4j-slf4j-impl</include>
>    <include>org.apache.logging.log4j:log4j-1.2-api</include>
> </includes> {code}
> i suggest to make the log4j to provided.
>  
> the assemble dependencySet scope just like filter 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to