Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Dogfood ImmutableFileCollection on production code (#4988)

This reverts commit 13eaebc2b1244511dcbff4c59cd41253e3b69642.

  1. … 87 more files in changeset.
Revert "Dogfood ImmutableFileCollection on production code (#4988)"

This reverts commit 834632674ca29b6fd190857947338b2b54a9bb62.

The commit caused a bug in incremental compilation, causing changes

to go undetected.

  1. … 87 more files in changeset.
Dogfood ImmutableFileCollection on production code (#4988)

Use ImmutableFileCollection in production code

  1. … 87 more files in changeset.
Allow worker actions to return a custom result

This will be necessary to transferm annotation

processing information from the compiler worker

back to the main daemon. The Compiler API already

returns a WorkResult, but until now this result

was discarded and a standard result instance

was returned instead.

This change allows worker actions to implement

Callable<WorkResult> and lets the compiler daemons

make use of this. In subsequent changes the compiler

will use a custom subclass that exposes more information.

This change does not affect the public WorkerExecutor API.

  1. … 3 more files in changeset.
Add `@Nullable` to `@Optional` task inputs

where applicable.

  1. … 46 more files in changeset.
Deprecated `PropertyState` interface, which is replaced by `Property`. Deprecated the `Project.property()`, `Script.property()` and `ProviderFactory.property()` methods, which are replaced by `ObjectFactory.property()`.

  1. … 19 more files in changeset.
Extracted public `Property` interface out of `PropertyState` as a first step in renaming. Changed other public APIs, except for the factory methods, to use `Property` instead of `PropertyState`.

Also moved factory method to create a `ListProperty<T>` from `ProviderFactory` to `ObjectFactory`.

  1. … 32 more files in changeset.
Replace CompileOptions.bootClasspath with FileCollection property

  1. … 10 more files in changeset.
Removed some unused type parameters.

  1. … 13 more files in changeset.
Removed the `workerImplementationClass` parameter from `WorkerFactory.getWorker()` as the implementation class is a detail of the worker factory and is a constant for a given worker factory.

  1. … 5 more files in changeset.
Move instantiator from spec to invocation

  1. … 16 more files in changeset.
Make compiler daemons sensitive to only BaseForkOptions

  1. … 9 more files in changeset.
Add missing exception message

Replace usages of org.gradle.api.Nullable

With javax.annotation.Nullable.

  1. … 460 more files in changeset.
Fix issue with duplicate compiler daemons and add reuse tests

  1. … 16 more files in changeset.
Fix setter selection in presence of multiple setters (#2386)

  1. … 5 more files in changeset.
Make worker daemons sensitive to all java fork options

    • -0
    • +40
    ./internal/tasks/compile/BaseForkOptionsConverter.java
  1. … 45 more files in changeset.
Compiler daemons use un-serialized execution specs

  1. … 4 more files in changeset.
Align compiler daemons with worker daemon infrastructure

  1. … 9 more files in changeset.
Refactor WorkerFactory so that the idle working dir is injected on the server side

  1. … 26 more files in changeset.
Remove setClasspath(Provider<FileCollection>)

A Provider for a FileCollection will be unnecessary soon, so we

shouldn't add it.

    • -15
    • +3
    ./tasks/compile/AbstractCompile.java
  1. … 2 more files in changeset.
Use providers instead of convention mapping

+review REVIEW-6502

    • -6
    • +35
    ./tasks/compile/AbstractCompile.java
  1. … 2 more files in changeset.
Do not track JVM memory settings as inputs to the task

+review REVIEW-6512

Added type parameter to `Worker` and related types, to describe what kinds of work the worker will accept.

  1. … 12 more files in changeset.
Changed `Worker` interface and worker daemon protocol to better separate the data (what to run) from the behaviour (how to run it). Now, all the data is included in the spec, and all the behaviour is included in the worker server implementation. Previously some of the data and behaviour also lived in the `WorkerAction` instance.

This change allows the worker server implementation, which is long running, better insight into the work to run so that it can, for example, inject long running services into the work.

Also moved the serialization of worker parameters so that it is done in the registration thread rather than the worker thread.

  1. … 17 more files in changeset.
Renamed AbstractDaemonCompiler to AbstractWorkerCompiler

And:

- DaemonJavaCompiler to WorkerJavaCompiler (supports both in-process and daemon workers)

- DaemonGroovyCompiler to WorkerGroovyCompiler (support both in-process and daemon workers)

- DaemonScalaCompiler to WorkerDaemonScalaCompiler (daemon workers only)

- DaemonPlayCompiler to WorkerDaemonPlayCompiler (daemon workers only)

    • -0
    • +100
    ./internal/tasks/compile/daemon/AbstractWorkerCompiler.java
  1. … 30 more files in changeset.
Non-forked Java compilation now uses in-process workers

Simplified AbstractDaemonCompiler and its type hierarchy.

Remove usage of some org.gradle.workers.internal types.

    • -0
    • +77
    ./internal/tasks/compile/ForkOptionsMerger.java
  1. … 9 more files in changeset.
Dogfood WorkerExecutor for java/groovy/scala compilation in worker daemons

Plus in-process groovy compilation

  1. … 29 more files in changeset.
Internal types, methods and variables renames in :workers

  1. … 63 more files in changeset.
Revert "Merge pull request #1486 from gradle/pm-parallel-in-process-workers"

This reverts commit 861bc80fe733f1ab6790860182a20957786e0711, reversing

changes made to cf20021050069a63b7ae629717eb663d95074997.

  1. … 23 more files in changeset.