7b48282ead7ca86abcf821accc312fadedc1431a,src/python/twitter/pants/tasks/jvm_compile/jvm_dependency_analyzer.py,JvmDependencyAnalyzer,_compute_targets_by_file,#JvmDependencyAnalyzer#,30

Before Change

            jarlibs_by_id[(jardep.org, jardep.name)].add(target)

    // Compute class -> target.
    classes_by_target = self._context.products.get_data("classes_by_target")
    for tgt, target_products in classes_by_target.items():
      for _, classes in target_products.abs_paths():
        for cls in classes:
          targets_by_file[cls].add(tgt)

    // Compute jar -> target.
    with Task.symlink_map_lock:
      all_symlinks_map = self._context.products.get_data("symlink_map").copy()
      // We make a copy, so it"s safe to use outside the lock.

After Change

              jarlibs_by_id[(jardep.org, jardep.name)].add(target)

    // Compute class -> target.
    with self._context.new_workunit(name="map_classes"):
      classes_by_target = self._context.products.get_data("classes_by_target")
      for tgt, target_products in classes_by_target.items():
        for _, classes in target_products.abs_paths():
          for cls in classes:
            targets_by_file[cls].add(tgt)

    // Compute jar -> target.
    with self._context.new_workunit(name="map_jars"):
      with Task.symlink_map_lock:
        all_symlinks_map = self._context.products.get_data("symlink_map").copy()
        // We make a copy, so it"s safe to use outside the lock.

Instances