Skip to content

Commit bd991a8

Browse files
JNSimbaclaude
andcommitted
[Feature] disable gz compression for json format in IT/E2E cases and fix checkstyle CI
Doris 2.1 does not support json compression, so disable default gz compression for json format test cases. Also fix checkstyle CI to install dependencies before running checkstyle:check. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 604430c commit bd991a8

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

.github/workflows/checkstyle.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040

4141
- name: Run java checkstyle
4242
run:
43-
cd flink-doris-connector && mvn clean checkstyle:check -Pflink1 -pl flink-doris-connector-flink1 -am
43+
cd flink-doris-connector && mvn clean install checkstyle:check -DskipTests -Pflink1 -pl flink-doris-connector-flink1 -am
4444

4545
- name: Setup java 17
4646
uses: actions/setup-java@v2
@@ -50,4 +50,4 @@ jobs:
5050

5151
- name: Run java checkstyle
5252
run:
53-
cd flink-doris-connector && mvn clean checkstyle:check -Pflink2 -pl flink-doris-connector-flink2 -am
53+
cd flink-doris-connector && mvn clean install checkstyle:check -DskipTests -Pflink2 -pl flink-doris-connector-flink2 -am

flink-doris-connector/flink-doris-connector-flink1/src/test/java/org/apache/doris/flink/container/AbstractE2EService.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,9 @@ protected void setSinkConfDefaultConfig(List<String> argList) {
142142
argList.add(JDBC_URL + "=" + getDorisQueryUrl());
143143
argList.add(SINK_CONF);
144144
argList.add(SINK_LABEL_PREFIX + "=" + "label");
145+
// disable gz compression for json format, doris 2.1 does not support json compression
146+
argList.add(SINK_CONF);
147+
argList.add("sink.properties.compress_type=");
145148
}
146149

147150
public static void closeE2EContainers() {

0 commit comments

Comments
 (0)