This is an automated email from the ASF dual-hosted git repository.
gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 69f4d00c43bb [SPARK-55958][BUILD][CONNECT] Remove unused
`add-scala-test-sources` setting from `pom.xml` in `connect-common`
69f4d00c43bb is described below
commit 69f4d00c43bb93e51edda5c92fde99020866b882
Author: Kousuke Saruta <[email protected]>
AuthorDate: Wed Mar 11 13:42:22 2026 +0900
[SPARK-55958][BUILD][CONNECT] Remove unused `add-scala-test-sources`
setting from `pom.xml` in `connect-common`
### What changes were proposed in this pull request?
This PR aims to remove unused `add-scala-test-sources` setting from
`pom.xml` in `connect-common`.
`add-scala-test-sources` adds `src/test/gen-java` as a test source
directory, but this directory has never existed in the `sql/connect/common`. It
appears to have been carried over from `sql/core/pom.xml` when the Connect
common module was originally created in SPARK-41369.
### Why are the changes needed?
Cleanup `pom.xml`.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
GA passed.
### Was this patch authored or co-authored using generative AI tooling?
Kiro CLI / Opus 4.6
Closes #54747 from sarutak/remove-unused-setting-from-connect-common.
Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
sql/connect/common/pom.xml | 18 ------------------
1 file changed, 18 deletions(-)
diff --git a/sql/connect/common/pom.xml b/sql/connect/common/pom.xml
index 550fe8b6b673..304a0aab14f5 100644
--- a/sql/connect/common/pom.xml
+++ b/sql/connect/common/pom.xml
@@ -107,24 +107,6 @@
<outputDirectory>target/scala-${scala.binary.version}/classes</outputDirectory>
<testOutputDirectory>target/scala-${scala.binary.version}/test-classes</testOutputDirectory>
<plugins>
- <plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>build-helper-maven-plugin</artifactId>
- <executions>
- <execution>
- <id>add-scala-test-sources</id>
- <phase>generate-test-sources</phase>
- <goals>
- <goal>add-test-source</goal>
- </goals>
- <configuration>
- <sources>
- <source>src/test/gen-java</source>
- </sources>
- </configuration>
- </execution>
- </executions>
- </plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
