Skip to content

Commit 9ad686f

Browse files
authored
Fix cachability of transport validation tasks (#136484) (#136506)
The transport version validation tasks were meant to be cachable, but since they don't output a file they aren't actually cachable. This commit fixes these tasks by making them PrecommitTasks which output a marker file on success.
1 parent 6f30995 commit 9ad686f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/ValidateTransportVersionReferencesTask.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
package org.elasticsearch.gradle.internal.transport;
1111

12-
import org.gradle.api.DefaultTask;
12+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitTask;
1313
import org.gradle.api.file.RegularFileProperty;
1414
import org.gradle.api.provider.Property;
1515
import org.gradle.api.services.ServiceReference;
@@ -30,7 +30,7 @@
3030
* Validates that each transport version reference has a referable definition.
3131
*/
3232
@CacheableTask
33-
public abstract class ValidateTransportVersionReferencesTask extends DefaultTask implements VerificationTask {
33+
public abstract class ValidateTransportVersionReferencesTask extends PrecommitTask implements VerificationTask {
3434

3535
@ServiceReference("transportVersionResources")
3636
abstract Property<TransportVersionResourcesService> getTransportResources();

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/ValidateTransportVersionResourcesTask.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@
1111

1212
import com.google.common.collect.Comparators;
1313

14+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitTask;
1415
import org.elasticsearch.gradle.internal.transport.TransportVersionResourcesService.IdAndDefinition;
15-
import org.gradle.api.DefaultTask;
1616
import org.gradle.api.file.ConfigurableFileCollection;
1717
import org.gradle.api.provider.Property;
1818
import org.gradle.api.services.ServiceReference;
@@ -40,7 +40,7 @@
4040
* Validates that each defined transport version constant is referenced by at least one project.
4141
*/
4242
@CacheableTask
43-
public abstract class ValidateTransportVersionResourcesTask extends DefaultTask implements VerificationTask {
43+
public abstract class ValidateTransportVersionResourcesTask extends PrecommitTask implements VerificationTask {
4444

4545
@InputDirectory
4646
@Optional

0 commit comments

Comments
 (0)