Skip to content

Commit 97ba00f

Browse files
committed
Split the log collector into log4j1 and log4j2; Avoid the jackson module in log collector.
1 parent 64d77d2 commit 97ba00f

File tree

60 files changed

+907
-454
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

60 files changed

+907
-454
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
<jdk.compile.version>1.8</jdk.compile.version>
5454
<maven.version>3.3.3</maven.version>
5555
<gson.version>2.8.5</gson.version>
56-
<fasterxml.jackson.version>2.11.3</fasterxml.jackson.version>
56+
<fasterxml.jackson.version>2.13.2</fasterxml.jackson.version>
5757
<math3.version>3.1.1</math3.version>
5858
<httpclient.version>4.5.4</httpclient.version>
5959
<httpmime.version>4.5.4</httpmime.version>

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/pom.xml renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/pom.xml

File renamed without changes.

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/main/assembly/package.xml renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/main/assembly/package.xml

File renamed without changes.

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkStreamisConfigAutowired.java renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkStreamisConfigAutowired.java

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.webank.wedatasphere.streamis.jobmanager.log.collector.flink;
22

33
import com.webank.wedatasphere.streamis.jobmanager.log.collector.config.StreamisLogAppenderConfig;
4+
import com.webank.wedatasphere.streamis.jobmanager.log.collector.log4j2.StreamisLog4j2AppenderConfig;
45
import com.webank.wedatasphere.streamis.jobmanager.plugin.StreamisConfigAutowired;
56
import org.apache.commons.lang3.StringUtils;
67
import org.apache.flink.configuration.Configuration;
@@ -49,17 +50,19 @@ public StreamisLogAppenderConfig logAppenderConfig(StreamisLogAppenderConfig.Bui
4950
gateway += this.configuration.getString(LOG_COLLECT_PATH, "/");
5051
builder.setRpcAddress(gateway);
5152
}
52-
List<String> filterStrategies = this.configuration.get(LOG_FILTER_STRATEGIES);
53-
for(String filterStrategy : filterStrategies){
54-
if ("LevelMatch".equals(filterStrategy)){
55-
builder.withFilter(LevelMatchFilter.newBuilder().setOnMatch(Filter.Result.ACCEPT).setOnMismatch(Filter.Result.DENY)
56-
.setLevel(Level.getLevel(this.configuration.getString(LOG_FILTER_LEVEL_MATCH))).build());
57-
}else if ("ThresholdFilter".equals(filterStrategy)){
58-
builder.withFilter(ThresholdFilter.createFilter(Level
59-
.getLevel(this.configuration.getString(LOG_FILTER_THRESHOLD_MATCH)), Filter.Result.ACCEPT, Filter.Result.DENY));
60-
}else if ("RegexMatch".equals(filterStrategy)){
61-
builder.withFilter(RegexFilter.createFilter( this.configuration.getString(LOG_FILTER_REGEX),
62-
null, true, Filter.Result.ACCEPT, Filter.Result.DENY));
53+
if (builder instanceof StreamisLog4j2AppenderConfig.Builder) {
54+
List<String> filterStrategies = this.configuration.get(LOG_FILTER_STRATEGIES);
55+
for (String filterStrategy : filterStrategies) {
56+
if ("LevelMatch".equals(filterStrategy)) {
57+
((StreamisLog4j2AppenderConfig.Builder)builder).withFilter(LevelMatchFilter.newBuilder().setOnMatch(Filter.Result.ACCEPT).setOnMismatch(Filter.Result.DENY)
58+
.setLevel(Level.getLevel(this.configuration.getString(LOG_FILTER_LEVEL_MATCH))).build());
59+
} else if ("ThresholdFilter".equals(filterStrategy)) {
60+
((StreamisLog4j2AppenderConfig.Builder)builder).withFilter(ThresholdFilter.createFilter(Level
61+
.getLevel(this.configuration.getString(LOG_FILTER_THRESHOLD_MATCH)), Filter.Result.ACCEPT, Filter.Result.DENY));
62+
} else if ("RegexMatch".equals(filterStrategy)) {
63+
((StreamisLog4j2AppenderConfig.Builder)builder).withFilter(RegexFilter.createFilter(this.configuration.getString(LOG_FILTER_REGEX),
64+
null, true, Filter.Result.ACCEPT, Filter.Result.DENY));
65+
}
6366
}
6467
}
6568
String hadoopUser = EnvironmentInformation.getHadoopUser();

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkStreamisConfigDefine.java renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkStreamisConfigDefine.java

File renamed without changes.

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/main/resources/META-INF/services/com.webank.wedatasphere.streamis.jobmanager.plugin.StreamisConfigAutowired renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/main/resources/META-INF/services/com.webank.wedatasphere.streamis.jobmanager.plugin.StreamisConfigAutowired

File renamed without changes.

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/test/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkConfigurationLoadTest.java renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/test/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/flink/FlinkConfigurationLoadTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
import org.apache.flink.configuration.GlobalConfiguration;
55
import org.junit.Test;
66

7-
import java.io.File;
87
import java.util.Enumeration;
98
import java.util.Objects;
109
import java.util.Properties;

streamis-jobmanager/streamis-job-log/flink-streamis-log-collector/src/test/resources/flink-conf.yaml renamed to streamis-jobmanager/streamis-job-log/job-log-collector/flink-streamis-log-collector/src/test/resources/flink-conf.yaml

File renamed without changes.
Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<project xmlns="http://maven.apache.org/POM/4.0.0"
3+
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
4+
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
5+
<parent>
6+
<artifactId>streamis-job-log</artifactId>
7+
<groupId>com.webank.wedatasphere.streamis</groupId>
8+
<version>0.2.0</version>
9+
</parent>
10+
<modelVersion>4.0.0</modelVersion>
11+
12+
<artifactId>streamis-job-log-collector-core</artifactId>
13+
14+
<properties>
15+
<maven.compiler.source>8</maven.compiler.source>
16+
<maven.compiler.target>8</maven.compiler.target>
17+
<httpclient.version>4.5.13</httpclient.version>
18+
<httpmine.version>4.5.4</httpmine.version>
19+
</properties>
20+
21+
<dependencies>
22+
<dependency>
23+
<groupId>com.webank.wedatasphere.streamis</groupId>
24+
<artifactId>streamis-job-log-common</artifactId>
25+
<version>${streamis.version}</version>
26+
</dependency>
27+
<!--http client module-->
28+
<dependency>
29+
<groupId>org.apache.httpcomponents</groupId>
30+
<artifactId>httpclient</artifactId>
31+
<version>${httpclient.version}</version>
32+
</dependency>
33+
<dependency>
34+
<groupId>org.apache.httpcomponents</groupId>
35+
<artifactId>httpmime</artifactId>
36+
<version>${httpmine.version}</version>
37+
</dependency>
38+
</dependencies>
39+
</project>

streamis-jobmanager/streamis-job-log/streamis-job-log-collector/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/ExceptionListener.java renamed to streamis-jobmanager/streamis-job-log/job-log-collector/streamis-job-log-collector-core/src/main/java/com/webank/wedatasphere/streamis/jobmanager/log/collector/ExceptionListener.java

File renamed without changes.

0 commit comments

Comments
 (0)