Skip to content

Commit

Permalink
[KYUUBI #6808] [BUILD]Replace systemProperties with systemPropertyVar…
Browse files Browse the repository at this point in the history
…iables in maven-surefire-plugin

# 🔍 Description
## Issue References 🔗

## Describe Your Solution 🔧
Fix the warning of `maven-surefire-plugin` that show `Parameter 'systemProperties' is deprecated. Use systemPropertyVariables instead.` in the build.

## Types of changes 🔖

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

#### Behavior Without This Pull Request ⚰️

#### Behavior With This Pull Request 🎉

#### Related Unit Tests

---

# Checklist 📝

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6808 from pionCham/use-systemPropertyVariables.

Closes #6808

668bfe2 [chengpeiming] Replace  systemProperties with systemPropertyVariables in maven-surefire-plugin
0e74b47 [chengpeiming] Revert "Replace systemProperties with systemPropertyVariables"
797384a [chengpeiming] Replace systemProperties with systemPropertyVariables

Authored-by: chengpeiming <[email protected]>
Signed-off-by: Bowen Liang <[email protected]>
(cherry picked from commit ad53eef)
Signed-off-by: Bowen Liang <[email protected]>
  • Loading branch information
pionCham authored and bowenliang123 committed Nov 13, 2024
1 parent 65cb1a4 commit ae62ec0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -1449,7 +1449,7 @@
<environmentVariables>
<KYUUBI_WORK_DIR_ROOT>${project.build.directory}/work</KYUUBI_WORK_DIR_ROOT>
</environmentVariables>
<systemProperties>
<systemPropertyVariables>
<log4j.ignoreTCL>true</log4j.ignoreTCL>
<log4j2.configurationFile>src/test/resources/log4j2-test.xml</log4j2.configurationFile>
<java.io.tmpdir>${project.build.directory}/tmp</java.io.tmpdir>
Expand All @@ -1459,7 +1459,7 @@
<sun.security.krb5.debug>false</sun.security.krb5.debug>
<kyuubi.operation.log.dir.root>${project.build.directory}/server_operation_logs</kyuubi.operation.log.dir.root>
<kyuubi.engine.operation.log.dir.root>${project.build.directory}/engine_operation_logs</kyuubi.engine.operation.log.dir.root>
</systemProperties>
</systemPropertyVariables>
</configuration>
</plugin>
<!-- enable scalatest -->
Expand Down

0 comments on commit ae62ec0

Please sign in to comment.