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

Change IStateManager.StartTrackingFromQuery to use ISnapshot instead of ValueBuffer #31117

Closed
Tracked by #26544
AndriySvyryd opened this issue Jun 21, 2023 · 0 comments · Fixed by #32099
Closed
Tracked by #26544
Labels
area-change-tracking area-perf area-query closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement

Comments

@AndriySvyryd
Copy link
Member

AndriySvyryd commented Jun 21, 2023

A new implementation of SnapshotFactoryFactory could return the expression that creates a snapshot instance based on the variable expressions corresponding to the shadow properties. This will allow it to be inlined into the shaper.

@ajcvickers ajcvickers self-assigned this Jul 7, 2023
@ajcvickers ajcvickers added this to the Backlog milestone Jul 7, 2023
AndriySvyryd added a commit that referenced this issue Oct 19, 2023
AndriySvyryd added a commit that referenced this issue Oct 19, 2023
AndriySvyryd added a commit that referenced this issue Oct 24, 2023
@ajcvickers ajcvickers modified the milestones: Backlog, 9.0.0 Nov 14, 2023
@ajcvickers ajcvickers added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Nov 14, 2023
@ajcvickers ajcvickers removed their assignment Nov 14, 2023
@ajcvickers ajcvickers modified the milestones: 9.0.0, 9.0.0-preview1 Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-change-tracking area-perf area-query closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants