pantsbuild / pants / 6255 / 3
0%
main: 93%

LAST BUILD BRANCH: perf/parse-build-files-once
DEFAULT BRANCH: main
Ran
Files 352
Run time 10s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 70.334% (+0.03%) from 70.308%
CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'"

push

travis-ci

gmalmquist 
Speeding up jvm-platform-validate step.

Using self.invalidated mechanism, targets are only re-evaluated if
they (1) they failed previously, (2) their dependencies have changed,
or (3) their platform -target version has changed.

Additionally, pruned down the list of dependencies that must be
considered for each target. Instead of evaluating all transitive
JvmTarget dependencies, only consider the "direct" JvmTarget
dependencies. "Direct" is in quotes because I'm defining a JvmTarget
'A' as "directly" depending on a JvmTarget 'B' iff there exists a
path in the dependency graph from 'A' to 'B', and there are no
internal vertices in the path that are JvmTargets.

This behavior is documented in the jvm_dependency_map property.

This addresses https://github.com/pantsbuild/pants/issues/1972. It'd probably be good for someone with access to test this change against the repos which were previously experiencing slow-down from jvm-platform-validate, to make sure this patch actually solves the problem.

Testing Done:
Manual testing implies caching is working as intended. Added a couple unit tests to test_jvm_platform_analysis.py. Everything passes locally.

CI went green: https://travis-ci.org/pantsbuild/pants/builds/75650551
CI went green: https://travis-ci.org/pantsbuild/pants/builds/75940854
CI went green: https://travis-ci.org/pantsbuild/pants/builds/76001873

Bugs closed: 1981

Reviewed at https://rbcommons.com/s/twitter/r/2626/

15024 of 21361 relevant lines covered (70.33%)

0.7 hits per line

Source Files on job 6255.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'")