[jira] [Commented] (FLINK-19900) Wrong property name for surefire log4j configuration

["Chesnay Schepler (Jira)" <ji...@apache.org>]

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

Chesnay Schepler commented on FLINK-19900:
------------------------------------------

We can just remove it; the log4j file is picked up automatically in both maven and the IDE, and on CI we explicitly use the correct log4j property.

> Wrong property name for surefire log4j configuration
> ----------------------------------------------------
>
>                 Key: FLINK-19900
>                 URL: https://issues.apache.org/jira/browse/FLINK-19900
>             Project: Flink
>          Issue Type: Bug
>          Components: Build System
>            Reporter: Dawid Wysakowicz
>            Assignee: Dawid Wysakowicz
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.12.0
>
>
> Log4j uses {{log4j.configurationFile}} system property for passing a configuration file. In our surefire configuration we use {{log4j.configuration}} property instead which has no effect.
> {code}
> <plugin>
> 	<groupId>org.apache.maven.plugins</groupId>
> 	<artifactId>maven-surefire-plugin</artifactId>
> 	<version>2.22.1</version>
> 	<configuration>
>          ....
> 		<systemPropertyVariables>
>                 ....
> 			<log4j.configuration>${log4j.configuration}</log4j.configuration>
> 		</systemPropertyVariables>
>                 ....
>          </configuration>
> {code}



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