Added a instrumentation of load calls #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

bbakerman merged 1 commit into master from load_intrumentation

Apr 2, 2025

Member

bbakerman commented Mar 11, 2025

The Insrtrumentation calls did not cover load calls. This adds that


          Added a instrumentation of load calls

325f82d

bbakerman added this to the 4.0.0 milestone

bbakerman commented

View reviewed changes

src/main/java/org/dataloader/instrumentation/DataLoaderInstrumentation.java

+                   */
+                  default DataLoaderInstrumentationContext<Object> beginLoad(DataLoader<?, ?> dataLoader, Object key, Object loadContext) {
+                      return null;
+                  }

Member Author

bbakerman Mar 11, 2025

This would allow some to "see" all DL load calls

andimarek approved these changes

View reviewed changes

bbakerman merged commit 212049c into master

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet