microsoft / FluidFramework

Library for building distributed, real-time collaborative web applications
https://fluidframework.com
MIT License
4.7k stars 528 forks source link

[GC] Remove used routes state from data store context #21939

Closed agarwal-navin closed 1 month ago

agarwal-navin commented 1 month ago

Currently, if a data store is not realized and updateUsedRoutes() is called on it, the used routes are saved and are passed on the child DDSes (and their summarizer nodes) during data store realization. This PR removes storing the used routes state from data store context. The summarizer node should pass this information directly to the child summarizer node for couple of reasons:

AB#3448

msfluid-bot commented 1 month ago
@fluid-example/bundle-size-tests: -35 Bytes
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 457.99 KB 457.95 KB -35 Bytes
azureClient.js 555.23 KB 555.21 KB -21 Bytes
connectionState.js 680 Bytes 680 Bytes No change
containerRuntime.js 258.67 KB 258.62 KB -56 Bytes
fluidFramework.js 406.94 KB 406.96 KB +14 Bytes
loader.js 134.04 KB 134.05 KB +14 Bytes
map.js 42.13 KB 42.14 KB +7 Bytes
matrix.js 146.41 KB 146.42 KB +7 Bytes
odspClient.js 523.37 KB 523.35 KB -21 Bytes
odspDriver.js 97.55 KB 97.57 KB +21 Bytes
odspPrefetchSnapshot.js 42.61 KB 42.62 KB +14 Bytes
sharedString.js 163.13 KB 163.14 KB +7 Bytes
sharedTree.js 397.46 KB 397.46 KB +7 Bytes
Total Size 3.3 MB 3.3 MB -35 Bytes

Baseline commit: 4c0c130f1df720694a1a1cab7fd0f22f792baac1

Generated by :no_entry_sign: dangerJS against 18101b8c1980847342e26a830886a714757b1f2c