-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Comparing changes
Open a pull request
base repository: graphql-java/graphql-java
base: validation-refactor
head repository: graphql-java/graphql-java
compare: master
- 20 commits
- 12 files changed
- 5 contributors
Commits on Feb 9, 2026
-
Bump aws-actions/configure-aws-credentials from 5 to 6
Bumps [aws-actions/configure-aws-credentials](https://github.com/aws-actions/configure-aws-credentials) from 5 to 6. - [Release notes](https://github.com/aws-actions/configure-aws-credentials/releases) - [Changelog](https://github.com/aws-actions/configure-aws-credentials/blob/main/CHANGELOG.md) - [Commits](aws-actions/configure-aws-credentials@v5...v6) --- updated-dependencies: - dependency-name: aws-actions/configure-aws-credentials dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for da5e6b1 - Browse repository at this point
Copy the full SHA da5e6b1View commit details -
Bump org.testng:testng from 7.11.0 to 7.12.0
Bumps [org.testng:testng](https://github.com/testng-team/testng) from 7.11.0 to 7.12.0. - [Release notes](https://github.com/testng-team/testng/releases) - [Changelog](https://github.com/testng-team/testng/blob/master/CHANGES.txt) - [Commits](testng-team/testng@7.11.0...7.12.0) --- updated-dependencies: - dependency-name: org.testng:testng dependency-version: 7.12.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 54f8b69 - Browse repository at this point
Copy the full SHA 54f8b69View commit details -
Bump com.fasterxml.jackson.core:jackson-databind from 2.20.1 to 2.21.0
Bumps [com.fasterxml.jackson.core:jackson-databind](https://github.com/FasterXML/jackson) from 2.20.1 to 2.21.0. - [Commits](https://github.com/FasterXML/jackson/commits) --- updated-dependencies: - dependency-name: com.fasterxml.jackson.core:jackson-databind dependency-version: 2.21.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 60bad37 - Browse repository at this point
Copy the full SHA 60bad37View commit details -
Bump net.ltgt.errorprone from 4.3.0 to 5.0.0
Bumps net.ltgt.errorprone from 4.3.0 to 5.0.0. --- updated-dependencies: - dependency-name: net.ltgt.errorprone dependency-version: 5.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for fd28e61 - Browse repository at this point
Copy the full SHA fd28e61View commit details
Commits on Feb 17, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 289505e - Browse repository at this point
Copy the full SHA 289505eView commit details
Commits on Feb 19, 2026
-
Configuration menu - View commit details
-
Copy full SHA for fd7ff0a - Browse repository at this point
Copy the full SHA fd7ff0aView commit details -
Avoid redundant array copy in Async$Many for synchronous execution path
In the all-synchronous execution path (no CompletableFutures), Async$Many allocated an Object[] to collect field values, then copied them into a new ArrayList in materialisedList(). Replace the copy with Arrays.asList() which wraps the existing array at zero cost. Benchmarked with a new ExecutionBenchmark (balanced tree: ~530 fields, ~2000 result scalars, depth 5) showing ~5% throughput improvement on the synchronous path. Also adds async-profiler support to build.gradle for JMH profiling. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for acc6c10 - Browse repository at this point
Copy the full SHA acc6c10View commit details -
Make ResultPath.toStringValue lazy and avoid intermediate string allo…
…cations The toString representation of ResultPath was eagerly computed in the constructor via initString(), but is never read during normal query execution — only used for error reporting. Make it lazy (computed on first toString() call) to eliminate all string work from the hot path. Also inline segmentToString() into initString() to avoid intermediate String allocations when the value is eventually computed, letting Java's StringConcatFactory handle it as a single multi-arg concat. Benchmarked ~30-78% throughput improvement vs master across all execution benchmarks. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 45d13fc - Browse repository at this point
Copy the full SHA 45d13fcView commit details -
Add fast-path DataFetcher lookup avoiding FieldCoordinates allocation
Every field fetch created a throwaway FieldCoordinates object just for a HashMap lookup. Add an internal nested Map<String, Map<String, ...>> (typeName → fieldName → factory) built at CodeRegistry construction time, and an internal getDataFetcher(String, String, GraphQLFieldDefinition) method that does the lookup by strings directly. Use this in ExecutionStrategy.fetchField to skip FieldCoordinates creation entirely. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 659fe34 - Browse repository at this point
Copy the full SHA 659fe34View commit details -
Merge pull request #4228 from graphql-java/validation-refactor
Consolidate validation rules into single OperationValidator and improve performance
Configuration menu - View commit details
-
Copy full SHA for f8f9892 - Browse repository at this point
Copy the full SHA f8f9892View commit details -
Merge remote-tracking branch 'origin/master' into async-many-material…
…ized-list-optimization
Configuration menu - View commit details
-
Copy full SHA for 9a54c18 - Browse repository at this point
Copy the full SHA 9a54c18View commit details -
Make fast-path getDataFetcher public and document performance benefit
Remove @internal from the String-based getDataFetcher overload and add proper javadoc documenting the ~54 KB/op allocation savings and 5-9% throughput improvement over the FieldCoordinates-based lookup. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 4011983 - Browse repository at this point
Copy the full SHA 4011983View commit details -
Simplify ResultPath.initString() — inlining is unnecessary with lazy …
…init Since toStringValue is lazily computed (once per path), the manual inlining of segmentToString() provides no measurable performance benefit. Simplify back to the clean delegation. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for c52498b - Browse repository at this point
Copy the full SHA c52498bView commit details -
Merge pull request #4252 from graphql-java/async-many-materialized-li…
…st-optimization Reduce allocations in Async$Many and ResultPath on execution hot path
Configuration menu - View commit details
-
Copy full SHA for 9ae83ab - Browse repository at this point
Copy the full SHA 9ae83abView commit details
Commits on Feb 20, 2026
-
Merge pull request #4241 from graphql-java/dependabot/github_actions/…
…aws-actions/configure-aws-credentials-6
Configuration menu - View commit details
-
Copy full SHA for 3d1e349 - Browse repository at this point
Copy the full SHA 3d1e349View commit details -
Merge pull request #4242 from graphql-java/dependabot/gradle/org.test…
…ng-testng-7.12.0
Configuration menu - View commit details
-
Copy full SHA for 7f0ded9 - Browse repository at this point
Copy the full SHA 7f0ded9View commit details -
Merge pull request #4243 from graphql-java/dependabot/gradle/com.fast…
…erxml.jackson.core-jackson-databind-2.21.0
Configuration menu - View commit details
-
Copy full SHA for 20d3645 - Browse repository at this point
Copy the full SHA 20d3645View commit details -
Merge pull request #4246 from graphql-java/dependabot/gradle/net.ltgt…
….errorprone-5.0.0
Configuration menu - View commit details
-
Copy full SHA for 6e7cb30 - Browse repository at this point
Copy the full SHA 6e7cb30View commit details
Commits on Feb 21, 2026
-
Fix SchemaTransformerBenchmark to pass schema validation
The benchmark's `infoDirective` lacked valid locations, causing schema validation failures during `transformSchema`. Additionally, the directive was not declared on the schema, so the post-transformation validation would reject it. This meant the benchmark was measuring error-path behavior rather than the real transformation cost. - Add FIELD_DEFINITION and OBJECT valid locations to `infoDirective` - Declare `infoDirective` as an additional directive on the schema - Import `DirectiveLocation` Co-Authored-By: Claude Opus 4.6 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for cb886dc - Browse repository at this point
Copy the full SHA cb886dcView commit details -
Configuration menu - View commit details
-
Copy full SHA for 50c22c1 - Browse repository at this point
Copy the full SHA 50c22c1View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff validation-refactor...master