microsoft / FluidFramework

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

ContaineRuntime: Refactor batch processing code to support either op-by-op or batch-all-at-once semantics #22501

Closed markfields closed 1 month ago

markfields commented 1 month ago

Description

Step one of AB#15221

No behavior change here, just refactoring to be able to express the result of RemoteMessageProcessor in terms of ops or batches.

Follow-up change to adjust the logic will be much more scoped then.

Reviewer Guidance

No behavior change. Notice that even in test files, the change is strictly refactoring, test results/asserts are unchanged.

msfluid-bot commented 1 month ago
@fluid-example/bundle-size-tests: +1.66 KB
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 459.95 KB 460.34 KB +398 Bytes
azureClient.js 557.88 KB 558.29 KB +412 Bytes
connectionState.js 680 Bytes 680 Bytes No change
containerRuntime.js 260.71 KB 261.08 KB +377 Bytes
fluidFramework.js 401.54 KB 401.55 KB +14 Bytes
loader.js 134.19 KB 134.2 KB +14 Bytes
map.js 42.43 KB 42.44 KB +7 Bytes
matrix.js 146.58 KB 146.58 KB +7 Bytes
odspClient.js 525.23 KB 525.64 KB +412 Bytes
odspDriver.js 97.8 KB 97.82 KB +21 Bytes
odspPrefetchSnapshot.js 42.76 KB 42.78 KB +14 Bytes
sharedString.js 163.3 KB 163.31 KB +7 Bytes
sharedTree.js 392 KB 392.01 KB +7 Bytes
Total Size 3.3 MB 3.3 MB +1.66 KB

Baseline commit: de91c3a6b2671e63d624ce60404e7312f111d1ce

Generated by :no_entry_sign: dangerJS against 32728d41a07ec9051fdb7ea0378de5dc7a10dfbb