Fix build after Result Document introduction#9094
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes build errors that occurred after the MetaDB merge by updating code to align with API changes and applying conditional compilation patterns consistently across command classes.
Changes:
- Added .NET 9.0 conditional compilation directives for
RequiresDynamicCodeandRequiresUnreferencedCodeattributes in OpenApiCommand and McpCommand classes - Updated JsonMemory.Rent() calls to include the required JsonMemoryKind.Json parameter
- Removed .NET 10 conditional compilation for null-forgiving operator in ServiceCollectionExtensions
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| OpenApiCommand.cs | Added NET9_0_OR_GREATER conditional compilation for attributes consistent with other command classes |
| McpCommand.cs | Added NET9_0_OR_GREATER conditional compilation for attributes consistent with other command classes |
| JsonResultEnumerable.cs | Updated JsonMemory.Rent() calls to include required JsonMemoryKind.Json parameter |
| ServiceCollectionExtensions.cs | Removed NET10 conditional compilation, standardizing on null-forgiving operator |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
🚀 Fusion Gateway Performance ResultsSimple Composite QueryConstant Load (50 VUs)
📊 Response Time Metrics
Ramping Load (0→50→500→50 VUs)
📊 Response Time Metrics
Executed Query fragment User on User {
id
username
name
}
fragment Review on Review {
id
body
}
fragment Product on Product {
inStock
name
price
shippingEstimate
upc
weight
}
query TestQuery {
topProducts(first: 5) {
...Product
reviews {
...Review
author {
...User
}
}
}
}Deep Recursion QueryConstant Load (50 VUs)
📊 Response Time Metrics
Ramping Load (0→50→500→50 VUs)
📊 Response Time Metrics
Executed Query fragment User on User {
id
username
name
}
fragment Review on Review {
id
body
}
fragment Product on Product {
inStock
name
price
shippingEstimate
upc
weight
}
query TestQuery {
users {
...User
reviews {
...Review
product {
...Product
reviews {
...Review
author {
...User
reviews {
...Review
product {
...Product
}
}
}
}
}
}
}
topProducts(first: 5) {
...Product
reviews {
...Review
author {
...User
reviews {
...Review
product {
...Product
}
}
}
}
}
}Variable Batching ThroughputConstant Load (50 VUs)
📊 Response Time Metrics
Ramping Load (0→50→500→50 VUs)
📊 Response Time Metrics
Executed Query query TestQuery_8f7a46ce_2(
$__fusion_1_upc: ID!
$__fusion_2_price: Long!
$__fusion_2_weight: Long!
) {
productByUpc(upc: $__fusion_1_upc) {
inStock
shippingEstimate(weight: $__fusion_2_weight, price: $__fusion_2_price)
}
}Variables (5 sets batched in single request) [
{ "__fusion_1_upc": "1", "__fusion_2_price": 899, "__fusion_2_weight": 100 },
{ "__fusion_1_upc": "2", "__fusion_2_price": 1299, "__fusion_2_weight": 1000 },
{ "__fusion_1_upc": "3", "__fusion_2_price": 15, "__fusion_2_weight": 20 },
{ "__fusion_1_upc": "4", "__fusion_2_price": 499, "__fusion_2_weight": 100 },
{ "__fusion_1_upc": "5", "__fusion_2_price": 1299, "__fusion_2_weight": 1000 }
]No baseline data available for comparison. Run 21756436297 • Commit 87d5fd9 • Fri, 06 Feb 2026 16:00:46 GMT |
No description provided.