perf: Only load node location when needed

Author: nikcio

performance/benchmark/BenchmarkDotNet.Artifacts/results/performance.Descriptions-report-github.md

@@ -10,11 +10,11 @@ Job=ShortRun  IterationCount=3  LaunchCount=1
 WarmupCount=3  
 
 ```
-| Method       | Mean           | Error         | StdDev       | Gen0        | Gen1       | Gen2      | Allocated     |
-|------------- |---------------:|--------------:|-------------:|------------:|-----------:|----------:|--------------:|
-| PetStoreYaml |       285.1 μs |     467.14 μs |     25.61 μs |      5.8594 |          - |         - |     361.38 KB |
-| PetStoreJson |       110.2 μs |      75.79 μs |      4.15 μs |      4.3945 |     0.9766 |         - |     223.52 KB |
-| GHESYaml     |   687,947.0 μs | 659,775.92 μs | 36,164.54 μs |   9000.0000 |  8000.0000 | 2000.0000 |  345336.55 KB |
-| GHESJson     |   270,583.7 μs | 134,378.11 μs |  7,365.72 μs |   4000.0000 |  3000.0000 | 1000.0000 |  206858.06 KB |
-| GHESNextYaml | 3,443,318.8 μs | 127,519.37 μs |  6,989.77 μs | 160000.0000 | 11000.0000 | 3000.0000 | 7701975.75 KB |
-| GHESNextJson | 2,954,715.0 μs | 327,737.12 μs | 17,964.38 μs | 155000.0000 | 19000.0000 | 1000.0000 | 7567167.19 KB |
+| Method       | Mean         | Error           | StdDev       | Gen0       | Gen1      | Gen2      | Allocated    |
+|------------- |-------------:|----------------:|-------------:|-----------:|----------:|----------:|-------------:|
+| PetStoreYaml |     266.8 μs |       423.33 μs |     23.20 μs |     5.8594 |         - |         - |    361.38 KB |
+| PetStoreJson |     104.9 μs |        38.83 μs |      2.13 μs |     4.3945 |    0.9766 |         - |    223.52 KB |
+| GHESYaml     | 627,326.7 μs |   525,719.23 μs | 28,816.45 μs |  9000.0000 | 8000.0000 | 2000.0000 | 345336.55 KB |
+| GHESJson     | 256,258.8 μs |   156,676.06 μs |  8,587.94 μs |  4000.0000 | 3000.0000 | 1000.0000 | 206858.38 KB |
+| GHESNextYaml | 840,612.7 μs | 1,063,489.38 μs | 58,293.44 μs | 20000.0000 | 9000.0000 | 2000.0000 | 908819.18 KB |
+| GHESNextJson | 478,093.3 μs |    98,044.22 μs |  5,374.13 μs | 16000.0000 | 7000.0000 | 1000.0000 | 774015.55 KB |

performance/benchmark/BenchmarkDotNet.Artifacts/results/performance.Descriptions-report.csv

@@ -1,7 +1,7 @@
 Method;Job;AnalyzeLaunchVariance;EvaluateOverhead;MaxAbsoluteError;MaxRelativeError;MinInvokeCount;MinIterationTime;OutlierMode;Affinity;EnvironmentVariables;Jit;LargeAddressAware;Platform;PowerPlanMode;Runtime;AllowVeryLargeObjects;Concurrent;CpuGroups;Force;HeapAffinitizeMask;HeapCount;NoAffinitize;RetainVm;Server;Arguments;BuildConfiguration;Clock;EngineFactory;NuGetReferences;Toolchain;IsMutator;InvocationCount;IterationCount;IterationTime;LaunchCount;MaxIterationCount;MaxWarmupIterationCount;MemoryRandomization;MinIterationCount;MinWarmupIterationCount;RunStrategy;UnrollFactor;WarmupCount;Mean;Error;StdDev;Gen0;Gen1;Gen2;Allocated
-PetStoreYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;285.1 μs;467.14 μs;25.61 μs;5.8594;0.0000;0.0000;361.38 KB
-PetStoreJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;110.2 μs;75.79 μs;4.15 μs;4.3945;0.9766;0.0000;223.52 KB
-GHESYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"687,947.0 μs";"659,775.92 μs";"36,164.54 μs";9000.0000;8000.0000;2000.0000;345336.55 KB
-GHESJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"270,583.7 μs";"134,378.11 μs";"7,365.72 μs";4000.0000;3000.0000;1000.0000;206858.06 KB
-GHESNextYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"3,443,318.8 μs";"127,519.37 μs";"6,989.77 μs";160000.0000;11000.0000;3000.0000;7701975.75 KB
-GHESNextJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"2,954,715.0 μs";"327,737.12 μs";"17,964.38 μs";155000.0000;19000.0000;1000.0000;7567167.19 KB
+PetStoreYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;266.8 μs;423.33 μs;23.20 μs;5.8594;0.0000;0.0000;361.38 KB
+PetStoreJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;104.9 μs;38.83 μs;2.13 μs;4.3945;0.9766;0.0000;223.52 KB
+GHESYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"627,326.7 μs";"525,719.23 μs";"28,816.45 μs";9000.0000;8000.0000;2000.0000;345336.55 KB
+GHESJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"256,258.8 μs";"156,676.06 μs";"8,587.94 μs";4000.0000;3000.0000;1000.0000;206858.38 KB
+GHESNextYaml;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"840,612.7 μs";"1,063,489.38 μs";"58,293.44 μs";20000.0000;9000.0000;2000.0000;908819.18 KB
+GHESNextJson;ShortRun;False;Default;Default;Default;Default;Default;Default;1111111111111111;Empty;RyuJit;Default;X64;8c5e7fda-e8bf-4a96-9a85-a6e23a8c635c;.NET 8.0;False;True;False;True;Default;Default;False;False;False;Default;Default;Default;Default;Default;Default;Default;Default;3;Default;1;Default;Default;Default;Default;Default;Default;16;3;"478,093.3 μs";"98,044.22 μs";"5,374.13 μs";16000.0000;7000.0000;1000.0000;774015.55 KB

performance/benchmark/BenchmarkDotNet.Artifacts/results/performance.Descriptions-report.html

@@ -2,7 +2,7 @@
 <html lang='en'>
 <head>
 <meta charset='utf-8' />
-<title>performance.Descriptions-20260222-171957</title>
+<title>performance.Descriptions-20260222-175619</title>
 
 <style type="text/css">
 	table { border-collapse: collapse; display: block; width: 100%; overflow: auto; }
@@ -24,14 +24,14 @@
 </code></pre>
 
 <table>
-<thead><tr><th>Method</th><th>Mean    </th><th>Error  </th><th>StdDev</th><th>Gen0 </th><th>Gen1</th><th>Gen2</th><th>Allocated</th>
+<thead><tr><th>Method</th><th>Mean  </th><th>Error    </th><th>StdDev</th><th>Gen0</th><th>Gen1</th><th>Gen2</th><th>Allocated</th>
 </tr>
-</thead><tbody><tr><td>PetStoreYaml</td><td>285.1 &mu;s</td><td>467.14 &mu;s</td><td>25.61 &mu;s</td><td>5.8594</td><td>-</td><td>-</td><td>361.38 KB</td>
-</tr><tr><td>PetStoreJson</td><td>110.2 &mu;s</td><td>75.79 &mu;s</td><td>4.15 &mu;s</td><td>4.3945</td><td>0.9766</td><td>-</td><td>223.52 KB</td>
-</tr><tr><td>GHESYaml</td><td>687,947.0 &mu;s</td><td>659,775.92 &mu;s</td><td>36,164.54 &mu;s</td><td>9000.0000</td><td>8000.0000</td><td>2000.0000</td><td>345336.55 KB</td>
-</tr><tr><td>GHESJson</td><td>270,583.7 &mu;s</td><td>134,378.11 &mu;s</td><td>7,365.72 &mu;s</td><td>4000.0000</td><td>3000.0000</td><td>1000.0000</td><td>206858.06 KB</td>
-</tr><tr><td>GHESNextYaml</td><td>3,443,318.8 &mu;s</td><td>127,519.37 &mu;s</td><td>6,989.77 &mu;s</td><td>160000.0000</td><td>11000.0000</td><td>3000.0000</td><td>7701975.75 KB</td>
-</tr><tr><td>GHESNextJson</td><td>2,954,715.0 &mu;s</td><td>327,737.12 &mu;s</td><td>17,964.38 &mu;s</td><td>155000.0000</td><td>19000.0000</td><td>1000.0000</td><td>7567167.19 KB</td>
+</thead><tbody><tr><td>PetStoreYaml</td><td>266.8 &mu;s</td><td>423.33 &mu;s</td><td>23.20 &mu;s</td><td>5.8594</td><td>-</td><td>-</td><td>361.38 KB</td>
+</tr><tr><td>PetStoreJson</td><td>104.9 &mu;s</td><td>38.83 &mu;s</td><td>2.13 &mu;s</td><td>4.3945</td><td>0.9766</td><td>-</td><td>223.52 KB</td>
+</tr><tr><td>GHESYaml</td><td>627,326.7 &mu;s</td><td>525,719.23 &mu;s</td><td>28,816.45 &mu;s</td><td>9000.0000</td><td>8000.0000</td><td>2000.0000</td><td>345336.55 KB</td>
+</tr><tr><td>GHESJson</td><td>256,258.8 &mu;s</td><td>156,676.06 &mu;s</td><td>8,587.94 &mu;s</td><td>4000.0000</td><td>3000.0000</td><td>1000.0000</td><td>206858.38 KB</td>
+</tr><tr><td>GHESNextYaml</td><td>840,612.7 &mu;s</td><td>1,063,489.38 &mu;s</td><td>58,293.44 &mu;s</td><td>20000.0000</td><td>9000.0000</td><td>2000.0000</td><td>908819.18 KB</td>
+</tr><tr><td>GHESNextJson</td><td>478,093.3 &mu;s</td><td>98,044.22 &mu;s</td><td>5,374.13 &mu;s</td><td>16000.0000</td><td>7000.0000</td><td>1000.0000</td><td>774015.55 KB</td>
 </tr></tbody></table>
 </body>
 </html>

performance/benchmark/BenchmarkDotNet.Artifacts/results/performance.Descriptions-report.json

@@ -366,10 +366,10 @@ public static IOpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocu
 
         var pointer = mapNode.GetReferencePointer();
         var identifier = mapNode.GetJsonSchemaIdentifier();
-        var nodeLocation = node.Context.GetLocation();
 
         if (pointer != null)
         {
+            var nodeLocation = node.Context.GetLocation();
             var reference = GetReferenceIdAndExternalResource(pointer);
             var result = new OpenApiSchemaReference(reference.Item1, hostDocument, reference.Item2);
             result.Reference.SetMetadataFromMapNode(mapNode);

src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs

@@ -366,10 +366,10 @@ public static IOpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocu
 
         var pointer = mapNode.GetReferencePointer();
         var identifier = mapNode.GetJsonSchemaIdentifier();
-        var nodeLocation = node.Context.GetLocation();
 
         if (pointer != null)
         {
+            var nodeLocation = node.Context.GetLocation();
             var reference = GetReferenceIdAndExternalResource(pointer);
             var result = new OpenApiSchemaReference(reference.Item1, hostDocument, reference.Item2);
             result.Reference.SetMetadataFromMapNode(mapNode);