Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rc.2 Port: Fix "savedOp" metadata property propagation for grouped ops (#20837) #20902

Conversation

anthony-murphy
Copy link
Contributor

Port of: #20837

Container class will set savedOp property on op before processing stashed ops. Container Runtime uses this information to differentiate ops.

I do not see us propagating savedOp metadata property from grouped ops (as visible in Container) to ungrouped ops. OpGroupingManager.ungroupOp() does not do it as far as I can see. I think we are getting lucky with ID compression ops as they are usually not grouped (there is usually a single ID compression op, as it's part of its own batch). When/if any of those assumptions change, this code would stop working.

Propagate properly this property.

…t#20837)

Container class will set savedOp property on op before processing stashed ops.
Container Runtime uses this information to differentiate ops.

I do not see us propagating savedOp metadata property from grouped ops (as visible in Container) to ungrouped ops. OpGroupingManager.ungroupOp() does not do it as far as I can see. I think we are getting lucky with ID compression ops as they are usually not grouped (there is usually a single ID compression op, as it's part of its own batch). When/if any of those assumptions change, this code would stop working.

Propagate properly this property.
@msfluid-bot
Copy link
Collaborator

@fluid-example/bundle-size-tests: +130 Bytes
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 448.33 KB 448.4 KB +65 Bytes
connectionState.js 680 Bytes 680 Bytes No change
containerRuntime.js 250.95 KB 251.01 KB +65 Bytes
loader.js 128.15 KB 128.15 KB No change
map.js 41.36 KB 41.36 KB No change
matrix.js 143.7 KB 143.7 KB No change
odspDriver.js 97.66 KB 97.66 KB No change
odspPrefetchSnapshot.js 42.36 KB 42.36 KB No change
sharedString.js 161.71 KB 161.71 KB No change
sharedTree.js 329.75 KB 329.75 KB No change
Total Size 1.75 MB 1.75 MB +130 Bytes

Baseline commit: f324866

Generated by 🚫 dangerJS against 400f91f

@anthony-murphy anthony-murphy merged commit 9c0167b into microsoft:release/client/2.0.0-rc.2.0 Apr 29, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants