Skip to content

Commit

Permalink
Fix log printing to correctly associate comp steps and numPackages wi…
Browse files Browse the repository at this point in the history
…th the

right log lines.  The variables for numLoadedPackages and computationSteps were
transposed in the logging logic.

PiperOrigin-RevId: 327143590
  • Loading branch information
kkress authored and copybara-github committed Aug 18, 2020
1 parent a7a0d48 commit 47943be
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -129,11 +129,11 @@ public synchronized void loadingFinished() {
c -> Durations.toMillis(c.getPackageMetricsInternal().getLoadDuration()));
logIfNonEmpty(
"Largest packages (num targets)",
packagesWithMostComputationSteps.getExtremeElements(),
largestPackages.getExtremeElements(),
c -> c.getPackageMetricsInternal().getNumTargets());
logIfNonEmpty(
"Packages with most computation steps",
largestPackages.getExtremeElements(),
packagesWithMostComputationSteps.getExtremeElements(),
c -> c.getPackageMetricsInternal().getComputationSteps());
logIfNonEmpty(
"Packages with most transitive loads (num bzl files)",
Expand Down

0 comments on commit 47943be

Please sign in to comment.