Merge pull request #4 from MPCoreDeveloper/master - #6
Merged
Conversation
Merge pull request #3 from MPCoreDeveloper/copilot/add-feature-time-t…
MPCoreDeveloper pushed a commit
that referenced
this pull request
Aug 31, 2026
…xed-width updates no longer append new versions UPDATE on the columnar (AppendOnlyEngine) path appended a new version per update (leaving stale rows for compaction) and re-pointed every index. Fixed-width rows (and variable-width rows whose stored length is unchanged) can be overwritten in place at their existing storage slot. - IStorageEngine.TryUpdateInPlace + AppendOnlyEngine/PageBasedEngine/PageBasedAdapter: overwrite the record at the reference when it fits (PageBased engines delegate to their existing in-place Update and report whether the reference stayed). - Storage.OverwriteRecordAt: writes [length][record] at the existing offset when the stored length is unchanged (identical encryption/magic-header handling to AppendBytes); returns false otherwise and is disabled inside transactions (append-only rollback). - Table.Update + UpdateMultiple columnar paths: try TryUpdateInPlace first; on success the reference and PK index stay valid and hash entries move in place (PK re-point only when the PK value changed); otherwise fall back to the append path. Tests: engine-level same-length (in-place, file does not grow) + different-length (falls back to append); SQL-level fixed-width 50x UPDATE file does not grow, and variable-width length change still appends (correctness unchanged). 1,635 tests green.
MPCoreDeveloper pushed a commit
that referenced
this pull request
Aug 31, 2026
…ed-width updates no longer append new versions
MPCoreDeveloper pushed a commit
that referenced
this pull request
Aug 31, 2026
…ds 5-8ms read-write open per statement (3.5-5.3k ops/s, 0 file growth)
MPCoreDeveloper pushed a commit
that referenced
this pull request
Aug 31, 2026
…s 5-8ms read-write open per statement (3.5-5.3k ops/s, 0 file growth)
MPCoreDeveloper pushed a commit
that referenced
this pull request
Aug 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge pull request #3 from MPCoreDeveloper/copilot/add-feature-time-t…