Giters

graphql-java / java-dataloader

A Java 8 port of Facebook DataLoader

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

the `synchronized` in DataLoader

dugenkui03 opened this issue · comments

commented

I guess the reason of usage synchronized is ensure the consistency of operation on loaderQueue,StatisticsCollector, CacheMap.

Remove the synchronized and make the concrete implementation of loaderQueue,StatisticsCollector, CacheMap to ensure the consistency, will be more flexible and can afford higher throughput.