694ad841ca4521a0996cd856c74a955405904443,src/python/twitter/pants/tasks/java_compile.py,JavaCompile,execute,#JavaCompile#Any#,117

Before Change

          cp.insert(0, (conf, self._classes_dir))

      with self.invalidated(java_targets, invalidate_dependants=True) as invalidated:
        if self._flatten:
          // The deps go to a single well-known file, so we need only pass in the invalid targets here.
          self.execute_single_compilation(invalidated.combined_invalid_versioned_targets(), cp)
        else:
          // We must pass all targets,even valid ones, to execute_single_compilation(), so it can
          // track the per-target deps correctly.
          for vt in invalidated.all_versioned_targets():
            self.execute_single_compilation(vt, cp)
            invalidated.update_versioned_target(vt)

      if self.context.products.isrequired("classes"):
        genmap = self.context.products.get("classes")

        // Map generated classes to the owning targets and sources.

After Change

          partition_size_hint=self._partition_size_hint) as invalidation_check:
        for vt in invalidation_check.all_vts:
          if vt.valid:  // Don"t compile, just post-process.
            self.post_process(vt)
        for vt in invalidation_check.invalid_vts_partitioned:
          // Compile, using partitions for efficiency.
          self.execute_single_compilation(vt, cp)
          if not self.dry_run:
            vt.update()

      if not self.dry_run:
        if self.context.products.isrequired("classes"):
          genmap = self.context.products.get("classes")

Instances