chained data loader performance improvements by andimarek · Pull Request #4122 · graphql-java/graphql-java

Conversation

@andimarek

Technically merged, but actually superseded by #4123

small tweak in PerLevelDataLoaderDispatchStrategy

@github-actions

Test Results

  326 files    326 suites   3m 29s ⏱️
5 022 tests 5 016 ✅ 6 💤 0 ❌
5 111 runs  5 105 ✅ 6 💤 0 ❌

Results for commit edeed73.

♻️ This comment has been updated with latest results.

bbakerman

private final Set<Integer> currentlyDelayedDispatchingLevels = ConcurrentHashMap.newKeySet();
private final Set<Integer> dispatchingStartedPerLevel = new HashSet<>();
private final Set<Integer> dispatchingFinishedPerLevel = new HashSet<>();
private final Set<Integer> currentlyDelayedDispatchingLevels = new HashSet<>();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are all these inside a lock?

@andimarek andimarek changed the title perf test for chained dataloader chained data loader performance improvements

Sep 22, 2025

2 participants

@andimarek @bbakerman