Java 8 forEachループ内でSpring Data JPAリポジトリを使用するのはスレッドセーフですか？

Question

Java 8 forEachループ内でSpring Data JPAリポジトリを使用できますか？スレッドセーフですか？

public interface MajorModOptRepository extends CrudRepository<MajorModOpt, Long> { 
    Set<MajorModOpt> findManyByManCodeAndModCode(String manCode, String modCode); 
} 

そしてここで、foreachループです：

@Component 
public class MajorModOptsHelper { 

    @Autowired 
    private MajorModOptRepository majorModOptRepository; 

    public void setMajorOpts(@NonNull List<Vehicle> vehicles) { 
     vehicles.forEach(this::setMajorOpts); 

    } 

    // This method is called by above forEach 
    public void setMajorOpts(Vehicle vehicle) { 
     ... 
     // Repository method is called here 
     Set<MajorModOptVO> knownOpts = majorModOptRepository.findManyByManCodeAndModCode(vehicle.getManCode(), vehicle.getModCode()); 
     ... 
    } 
} 

は、事前にありがとうございます。

Answer 1

トレッドセーブです。マルチスレッドは必要ありません。

あなたが気にするべき唯一の事柄は、それぞれfindManyByManCodeAndModCode（あなたがそれを望む場合を除く）ごとに新しい取引を開くことではありません。これを解決する最も簡単な方法はをsetMajorOpts(@NonNull List<Vehicle> vehicles)メソッドに追加することです。

Answer 2

現在のforEach実装または従来のfor-loopでは、コードはスレッドセーフです。

あなたは概念的に次の2つのメソッドにList<Vehicle>save操作を分割しているので、（プログラムまたは@Transactional境界を介して実行）の代わりにトランザクション境界を心配する必要があります。 Vehiclesaveのいずれかの操作が失敗した場合はどうなりますか？ Propagation.REQUIREDのデフォルト値では、操作全体がロールバックされますが、2台目の車両saveの操作がPropagation.REQUIRES_NEWの場合、前のすべての事前障害Vehicleはすべてコミット（=コミット）されています。