Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: graphql-java/graphql-java
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: graphql-java/graphql-java
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: fix/4366-profiler-unwired-methods
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 1 commit
  • 3 files changed
  • 1 contributor

Commits on May 24, 2026

  1. Wire up 3 of 4 unwired Profiler methods for DataLoader tracking (#4366)

    The Profiler interface declared dataLoaderUsed, batchLoadedOldStrategy,
    and batchLoadedNewStrategy methods that were never called, leaving
    ProfilerResult.getDataLoaderLoadInvocations() and getDispatchEvents()
    always empty.
    
    - Call profiler.dataLoaderUsed() on each DataLoader load/loadMany
    - Call profiler.batchLoadedOldStrategy() per-DataLoader in non-chaining dispatch
    - Call profiler.batchLoadedNewStrategy() per-DataLoader in chaining dispatch
    - Track DataLoader names through ChainedDLStack for new strategy reporting
    - Enable previously-ignored profiler tests for DataLoader tracking
    
    https://claude.ai/code/session_01ERFZobxF9aVgfU3L86x28r
    claude committed May 24, 2026
    Configuration menu
    Copy the full SHA
    4c7c755 View commit details
    Browse the repository at this point in the history
Loading