You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datafusion.apache.org by gi...@apache.org on 2026/09/21 20:01:48 UTC
(datafusion-comet) branch gh-readonly-queue/main/pr-6084-09b44ad6fa17f58f4bbccf958c5cf02d790f7334 created (now ccdff0a4bf)
This is an automated email from the ASF dual-hosted git repository.
github-merge-queue[bot] pushed a change to branch gh-readonly-queue/main/pr-6084-09b44ad6fa17f58f4bbccf958c5cf02d790f7334
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git
at ccdff0a4bf chore: remove JaCoCo from the build (#6084)
This branch includes the following new commits:
new ccdff0a4bf chore: remove JaCoCo from the build (#6084)
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datafusion.apache.org
For additional commands, e-mail: commits-help@datafusion.apache.org
(datafusion-comet) 01/01: chore: remove JaCoCo from the build (#6084)
Posted by gi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
github-merge-queue[bot] pushed a commit to branch gh-readonly-queue/main/pr-6084-09b44ad6fa17f58f4bbccf958c5cf02d790f7334
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git
commit ccdff0a4bfeee377ffa8654a1ef7843e63b3b8b8
Author: Andy Grove <ag...@apache.org>
AuthorDate: Mon Sep 21 20:01:26 2026 +0000
chore: remove JaCoCo from the build (#6084)
JaCoCo was added in #163 to publish JVM coverage to Codecov. #3381 removed
the codecov upload step, which was the only consumer, but left the plugin
in place. Since then every build has instrumented every forked test JVM and
generated a report that nothing reads: no CI step uploads it, no check rule
gates on it, and no path in the repo references it.
It was never configured to measure much either. There is no report-aggregate,
so each module reports against its own exec file; common has no test sources
of its own, so its report is always skipped even though its classes are
exercised by the spark module's tests.
Also drop "including coverage" from the upload-test-reports description in
the java-test action, which has been stale since #3381 -- the step it guards
uploads surefire .txt reports only.
---
.github/actions/java-test/action.yaml | 2 +-
pom.xml | 25 -------------------------
2 files changed, 1 insertion(+), 26 deletions(-)
diff --git a/.github/actions/java-test/action.yaml b/.github/actions/java-test/action.yaml
index 36dd15f580..78eebf40e8 100644
--- a/.github/actions/java-test/action.yaml
+++ b/.github/actions/java-test/action.yaml
@@ -30,7 +30,7 @@ inputs:
required: false
default: ''
upload-test-reports:
- description: 'Whether to upload test results including coverage to GitHub'
+ description: 'Whether to upload test results to GitHub'
required: false
default: 'false'
skip-native-build:
diff --git a/pom.xml b/pom.xml
index 3288c69fa7..23a3c08bb2 100644
--- a/pom.xml
+++ b/pom.xml
@@ -89,7 +89,6 @@ under the License.
<arrow.version>18.3.0</arrow.version>
<codehaus.jackson.version>1.9.13</codehaus.jackson.version>
<spotless.version>2.43.0</spotless.version>
- <jacoco.version>0.8.11</jacoco.version>
<semanticdb.version>4.13.6</semanticdb.version>
<slf4j.version>2.0.17</slf4j.version>
<guava.version>33.2.1-jre</guava.version>
@@ -1041,11 +1040,6 @@ under the License.
<artifactId>flatten-maven-plugin</artifactId>
<version>${flatten-maven-plugin.version}</version>
</plugin>
- <plugin>
- <groupId>org.jacoco</groupId>
- <artifactId>jacoco-maven-plugin</artifactId>
- <version>${jacoco.version}</version>
- </plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>build-helper-maven-plugin</artifactId>
@@ -1304,25 +1298,6 @@ under the License.
</execution>
</executions>
</plugin>
- <plugin>
- <groupId>org.jacoco</groupId>
- <artifactId>jacoco-maven-plugin</artifactId>
- <executions>
- <execution>
- <id>default-prepare-agent</id>
- <goals>
- <goal>prepare-agent</goal>
- </goals>
- </execution>
- <execution>
- <id>report</id>
- <phase>test</phase>
- <goals>
- <goal>report</goal>
- </goals>
- </execution>
- </executions>
- </plugin>
</plugins>
</build>
</project>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datafusion.apache.org
For additional commands, e-mail: commits-help@datafusion.apache.org