This is an automated email from the ASF dual-hosted git repository.
zihaoxiang pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git
The following commit(s) were added to refs/heads/dev by this push:
new 291058adc7 [Improvement-18056] Remove unused code in storage module
(#18114)
291058adc7 is described below
commit 291058adc7f068f2c0897a1953d250c7725fc365
Author: xiangzihao <[email protected]>
AuthorDate: Wed Apr 1 16:43:50 2026 +0800
[Improvement-18056] Remove unused code in storage module (#18114)
---
.github/workflows/unit-test.yml | 6 +--
.../plugin/storage/abs/AbsStorageOperator.java | 2 +-
.../src/test/resources/logback.xml | 21 ----------
.../plugin/storage/api/StorageType.java | 46 ++++++----------------
.../storage/api/constants/DateConstants.java | 46 ----------------------
.../storage/api/constants/StorageConstants.java | 2 -
.../plugin/storage/cos/CosStorageOperator.java | 2 +-
.../src/test/resources/logback.xml | 21 ----------
.../src/test/resources/logback.xml | 21 ----------
.../src/test/resources/storage/hello.sh | 18 ---------
.../src/test/resources/storage/sql/demo.sql | 18 ---------
.../plugin/storage/obs/ObsStorageOperator.java | 2 +-
.../src/test/resources/logback.xml | 21 ----------
.../plugin/storage/oss/OssStorageOperator.java | 40 +++----------------
.../src/test/resources/logback.xml | 21 ----------
.../src/test/resources/student/student.sql | 18 ---------
16 files changed, 23 insertions(+), 282 deletions(-)
diff --git a/.github/workflows/unit-test.yml b/.github/workflows/unit-test.yml
index ff6641520b..8e0f309dc3 100644
--- a/.github/workflows/unit-test.yml
+++ b/.github/workflows/unit-test.yml
@@ -171,7 +171,7 @@ jobs:
name: SonarCloud Analysis
runs-on: ubuntu-latest
needs: [ paths-filter, unit-test ]
- if: ${{ always() && (needs.paths-filter.outputs.not-ignore == 'true' ||
github.event_name == 'push') }}
+ if: ${{ needs.unit-test.result == 'success' }}
timeout-minutes: 30
steps:
- uses: actions/checkout@v6
@@ -250,8 +250,8 @@ jobs:
publish-test-results:
name: Publish Test Results
runs-on: ubuntu-latest
- needs: unit-test
- if: always()
+ needs: [ paths-filter, unit-test ]
+ if: ${{ needs.unit-test.result == 'success' }}
steps:
- name: Download all surefire reports
uses: actions/download-artifact@v8
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/main/java/org/apache/dolphinscheduler/plugin/storage/abs/AbsStorageOperator.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/main/java/org/apache/dolphinscheduler/plugin/storage/abs/AbsStorageOperator.java
index edd2b86c59..8921507fa7 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/main/java/org/apache/dolphinscheduler/plugin/storage/abs/AbsStorageOperator.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/main/java/org/apache/dolphinscheduler/plugin/storage/abs/AbsStorageOperator.java
@@ -183,7 +183,7 @@ public class AbsStorageOperator extends
AbstractStorageOperator implements Close
return null;
}
- public void checkContainerNameExists(String containerName) {
+ private void checkContainerNameExists(String containerName) {
if (StringUtils.isBlank(containerName)) {
throw new IllegalArgumentException(containerName + " is blank");
}
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/test/resources/logback.xml
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/test/resources/logback.xml
deleted file mode 100644
index 6f211959c5..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-abs/src/test/resources/logback.xml
+++ /dev/null
@@ -1,21 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
- ~ Licensed to the Apache Software Foundation (ASF) under one or more
- ~ contributor license agreements. See the NOTICE file distributed with
- ~ this work for additional information regarding copyright ownership.
- ~ The ASF licenses this file to You under the Apache License, Version 2.0
- ~ (the "License"); you may not use this file except in compliance with
- ~ the License. You may obtain a copy of the License at
- ~
- ~ http://www.apache.org/licenses/LICENSE-2.0
- ~
- ~ Unless required by applicable law or agreed to in writing, software
- ~ distributed under the License is distributed on an "AS IS" BASIS,
- ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- ~ See the License for the specific language governing permissions and
- ~ limitations under the License.
- -->
-
-<configuration>
- <logger name="*" level="ERROR"/>
-</configuration>
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/StorageType.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/StorageType.java
index bd56113839..5262f4d267 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/StorageType.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/StorageType.java
@@ -21,42 +21,20 @@ import java.util.Optional;
public enum StorageType {
- LOCAL(0, "LOCAL"),
- HDFS(1, "HDFS"),
- OSS(2, "OSS"),
- S3(3, "S3"),
- GCS(4, "GCS"),
-
- ABS(5, "ABS"),
-
- OBS(6, "OBS"),
-
- COS(7, "COS"),
-
- ;
-
- private final int code;
- private final String name;
-
- StorageType(int code, String name) {
- this.code = code;
- this.name = name;
- }
-
- public int getCode() {
- return code;
- }
-
- public String getName() {
- return name;
- }
+ LOCAL,
+ HDFS,
+ OSS,
+ S3,
+ GCS,
+ ABS,
+ OBS,
+ COS;
public static Optional<StorageType> getStorageType(String name) {
- for (StorageType storageType : StorageType.values()) {
- if (storageType.getName().equals(name)) {
- return Optional.of(storageType);
- }
+ try {
+ return Optional.of(StorageType.valueOf(name));
+ } catch (IllegalArgumentException | NullPointerException ex) {
+ return Optional.empty();
}
- return Optional.empty();
}
}
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/DateConstants.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/DateConstants.java
deleted file mode 100644
index 41473edf5a..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/DateConstants.java
+++ /dev/null
@@ -1,46 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.apache.dolphinscheduler.plugin.storage.api.constants;
-
-import lombok.experimental.UtilityClass;
-
-@UtilityClass
-public class DateConstants {
-
- public static final String YYYY_MM_DD_HH_MM_SS = "yyyy-MM-dd HH:mm:ss";
-
- public static final String YYYYMMDDHHMMSSSSS = "yyyyMMddHHmmssSSS";
-
- public static final String YYYYMMDD = "yyyyMMdd";
-
- /**
- * system date(yyyyMMddHHmmss)
- */
- public static final String PARAMETER_DATETIME = "system.datetime";
-
- /**
- * system date(yyyymmdd) today
- */
- public static final String PARAMETER_CURRENT_DATE = "system.biz.curdate";
-
- /**
- * system date(yyyymmdd) yesterday
- */
- public static final String PARAMETER_BUSINESS_DATE = "system.biz.date";
-
-}
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/StorageConstants.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/StorageConstants.java
index 34189de481..576ebe057c 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/StorageConstants.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-api/src/main/java/org/apache/dolphinscheduler/plugin/storage/api/constants/StorageConstants.java
@@ -22,8 +22,6 @@ import lombok.experimental.UtilityClass;
@UtilityClass
public final class StorageConstants {
- public static final String RESOURCE_TYPE_FILE = "resources";
-
/**
* resource.hdfs.fs.defaultFS
*/
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/main/java/org/apache/dolphinscheduler/plugin/storage/cos/CosStorageOperator.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/main/java/org/apache/dolphinscheduler/plugin/storage/cos/CosStorageOperator.java
index 037fcbeea3..d8fdb635c2 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/main/java/org/apache/dolphinscheduler/plugin/storage/cos/CosStorageOperator.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/main/java/org/apache/dolphinscheduler/plugin/storage/cos/CosStorageOperator.java
@@ -307,7 +307,7 @@ public class CosStorageOperator extends
AbstractStorageOperator implements Close
return transformCOSObjectToStorageEntity(object);
}
- public void ensureBucketSuccessfullyCreated(String bucketName) {
+ private void ensureBucketSuccessfullyCreated(String bucketName) {
if (StringUtils.isBlank(bucketName)) {
throw new
IllegalArgumentException(CosStorageConstants.TENCENT_CLOUD_COS_BUCKET_NAME + "
is empty");
}
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/test/resources/logback.xml
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/test/resources/logback.xml
deleted file mode 100644
index 6f211959c5..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-cos/src/test/resources/logback.xml
+++ /dev/null
@@ -1,21 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
- ~ Licensed to the Apache Software Foundation (ASF) under one or more
- ~ contributor license agreements. See the NOTICE file distributed with
- ~ this work for additional information regarding copyright ownership.
- ~ The ASF licenses this file to You under the Apache License, Version 2.0
- ~ (the "License"); you may not use this file except in compliance with
- ~ the License. You may obtain a copy of the License at
- ~
- ~ http://www.apache.org/licenses/LICENSE-2.0
- ~
- ~ Unless required by applicable law or agreed to in writing, software
- ~ distributed under the License is distributed on an "AS IS" BASIS,
- ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- ~ See the License for the specific language governing permissions and
- ~ limitations under the License.
- -->
-
-<configuration>
- <logger name="*" level="ERROR"/>
-</configuration>
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-gcs/src/test/resources/logback.xml
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-gcs/src/test/resources/logback.xml
deleted file mode 100644
index 6f211959c5..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-gcs/src/test/resources/logback.xml
+++ /dev/null
@@ -1,21 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
- ~ Licensed to the Apache Software Foundation (ASF) under one or more
- ~ contributor license agreements. See the NOTICE file distributed with
- ~ this work for additional information regarding copyright ownership.
- ~ The ASF licenses this file to You under the Apache License, Version 2.0
- ~ (the "License"); you may not use this file except in compliance with
- ~ the License. You may obtain a copy of the License at
- ~
- ~ http://www.apache.org/licenses/LICENSE-2.0
- ~
- ~ Unless required by applicable law or agreed to in writing, software
- ~ distributed under the License is distributed on an "AS IS" BASIS,
- ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- ~ See the License for the specific language governing permissions and
- ~ limitations under the License.
- -->
-
-<configuration>
- <logger name="*" level="ERROR"/>
-</configuration>
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/hello.sh
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/hello.sh
deleted file mode 100644
index 9ee3583189..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/hello.sh
+++ /dev/null
@@ -1,18 +0,0 @@
-#!/bin/bash
-#
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements. See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-echo hello
\ No newline at end of file
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/sql/demo.sql
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/sql/demo.sql
deleted file mode 100644
index 83ebbac8d0..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-hdfs/src/test/resources/storage/sql/demo.sql
+++ /dev/null
@@ -1,18 +0,0 @@
-#
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements. See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-select * from t;
\ No newline at end of file
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/main/java/org/apache/dolphinscheduler/plugin/storage/obs/ObsStorageOperator.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/main/java/org/apache/dolphinscheduler/plugin/storage/obs/ObsStorageOperator.java
index fd96ff16a0..b3cfca0f09 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/main/java/org/apache/dolphinscheduler/plugin/storage/obs/ObsStorageOperator.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/main/java/org/apache/dolphinscheduler/plugin/storage/obs/ObsStorageOperator.java
@@ -233,7 +233,7 @@ public class ObsStorageOperator extends
AbstractStorageOperator implements Close
return transformObsObjectToStorageEntity(object);
}
- public void ensureBucketSuccessfullyCreated(String bucketName) {
+ private void ensureBucketSuccessfullyCreated(String bucketName) {
if (StringUtils.isBlank(bucketName)) {
throw new
IllegalArgumentException("resource.alibaba.cloud.obs.bucket.name is empty");
}
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/test/resources/logback.xml
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/test/resources/logback.xml
deleted file mode 100644
index 6f211959c5..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-obs/src/test/resources/logback.xml
+++ /dev/null
@@ -1,21 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
- ~ Licensed to the Apache Software Foundation (ASF) under one or more
- ~ contributor license agreements. See the NOTICE file distributed with
- ~ this work for additional information regarding copyright ownership.
- ~ The ASF licenses this file to You under the Apache License, Version 2.0
- ~ (the "License"); you may not use this file except in compliance with
- ~ the License. You may obtain a copy of the License at
- ~
- ~ http://www.apache.org/licenses/LICENSE-2.0
- ~
- ~ Unless required by applicable law or agreed to in writing, software
- ~ distributed under the License is distributed on an "AS IS" BASIS,
- ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- ~ See the License for the specific language governing permissions and
- ~ limitations under the License.
- -->
-
-<configuration>
- <logger name="*" level="ERROR"/>
-</configuration>
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/main/java/org/apache/dolphinscheduler/plugin/storage/oss/OssStorageOperator.java
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/main/java/org/apache/dolphinscheduler/plugin/storage/oss/OssStorageOperator.java
index dd4c404e20..de8c12e470 100644
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/main/java/org/apache/dolphinscheduler/plugin/storage/oss/OssStorageOperator.java
+++
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/main/java/org/apache/dolphinscheduler/plugin/storage/oss/OssStorageOperator.java
@@ -62,18 +62,10 @@ import com.aliyun.oss.model.PutObjectRequest;
@Slf4j
public class OssStorageOperator extends AbstractStorageOperator implements
Closeable, StorageOperator {
- private String accessKeyId;
-
- private String accessKeySecret;
-
private String region;
private String bucketName;
- private String endPoint;
-
- private OssConnection ossConnection;
-
private OSS ossClient;
public OssStorageOperator(String resourceBaseAbsolutePath) {
@@ -82,25 +74,12 @@ public class OssStorageOperator extends
AbstractStorageOperator implements Close
}
private void init() {
- this.accessKeyId = readOssAccessKeyID();
- this.accessKeySecret = readOssAccessKeySecret();
- this.endPoint = readOssEndPoint();
- this.region = readOssRegion();
- this.bucketName = readOssBucketName();
- this.ossConnection = buildOssConnection();
- this.ossClient = buildOssClient();
- ensureBucketSuccessfullyCreated(bucketName);
- }
-
- // TODO: change to use the following init method after DS supports
Configuration / Connection Center
- public void init(OssConnection ossConnection) {
- this.accessKeyId = readOssAccessKeyID();
- this.accessKeySecret = readOssAccessKeySecret();
- this.endPoint = readOssEndPoint();
+ final String accessKeyId = readOssAccessKeyID();
+ final String accessKeySecret = readOssAccessKeySecret();
+ final String endPoint = readOssEndPoint();
this.region = readOssRegion();
this.bucketName = readOssBucketName();
- this.ossConnection = ossConnection;
- this.ossClient = buildOssClient();
+ this.ossClient = OssClientFactory.buildOssClient(new
OssConnection(accessKeyId, accessKeySecret, endPoint));
ensureBucketSuccessfullyCreated(bucketName);
}
@@ -124,10 +103,6 @@ public class OssStorageOperator extends
AbstractStorageOperator implements Close
return
PropertyUtils.getString(StorageConstants.ALIBABA_CLOUD_OSS_END_POINT);
}
- protected OssConnection buildOssConnection() {
- return new OssConnection(accessKeyId, accessKeySecret, endPoint);
- }
-
@Override
public String getStorageBaseDirectory() {
// All directory should end with File.separator
@@ -311,7 +286,7 @@ public class OssStorageOperator extends
AbstractStorageOperator implements Close
return transformOSSObjectToStorageEntity(object);
}
- public void ensureBucketSuccessfullyCreated(String bucketName) {
+ private void ensureBucketSuccessfullyCreated(String bucketName) {
if (StringUtils.isBlank(bucketName)) {
throw new
IllegalArgumentException("resource.alibaba.cloud.oss.bucket.name is empty");
}
@@ -324,11 +299,6 @@ public class OssStorageOperator extends
AbstractStorageOperator implements Close
log.info("bucketName: {} has been found, the current regionName is
{}", bucketName, region);
}
-
- protected OSS buildOssClient() {
- return OssClientFactory.buildOssClient(ossConnection);
- }
-
protected StorageEntity transformOSSObjectToStorageEntity(OSSObject
ossObject) {
ResourceMetadata resourceMetaData =
getResourceMetaData(ossObject.getKey());
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/test/resources/logback.xml
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/test/resources/logback.xml
deleted file mode 100644
index 6f211959c5..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-oss/src/test/resources/logback.xml
+++ /dev/null
@@ -1,21 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
- ~ Licensed to the Apache Software Foundation (ASF) under one or more
- ~ contributor license agreements. See the NOTICE file distributed with
- ~ this work for additional information regarding copyright ownership.
- ~ The ASF licenses this file to You under the Apache License, Version 2.0
- ~ (the "License"); you may not use this file except in compliance with
- ~ the License. You may obtain a copy of the License at
- ~
- ~ http://www.apache.org/licenses/LICENSE-2.0
- ~
- ~ Unless required by applicable law or agreed to in writing, software
- ~ distributed under the License is distributed on an "AS IS" BASIS,
- ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- ~ See the License for the specific language governing permissions and
- ~ limitations under the License.
- -->
-
-<configuration>
- <logger name="*" level="ERROR"/>
-</configuration>
diff --git
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-s3/src/test/resources/student/student.sql
b/dolphinscheduler-storage-plugin/dolphinscheduler-storage-s3/src/test/resources/student/student.sql
deleted file mode 100644
index 6eab26f880..0000000000
---
a/dolphinscheduler-storage-plugin/dolphinscheduler-storage-s3/src/test/resources/student/student.sql
+++ /dev/null
@@ -1,18 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
-*/
-
-select * from t_ds_student;
\ No newline at end of file
