Skip to content

deprecation: limit deep unmarking for better performance#38082

Merged
DanielMSchmidt merged 14 commits intomainfrom
limit-usage-of-deep-unmarking-2
Feb 3, 2026
Merged

deprecation: limit deep unmarking for better performance#38082
DanielMSchmidt merged 14 commits intomainfrom
limit-usage-of-deep-unmarking-2

Conversation

@DanielMSchmidt
Copy link
Contributor

@DanielMSchmidt DanielMSchmidt commented Jan 21, 2026

PR Chain

Unmarking deep is an expensive operation that we want to circumvent if possible. This PR tinkers with the API so that less unmarking is necessary.

Fixes #

Target Release

1.15.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@DanielMSchmidt DanielMSchmidt requested a review from a team as a code owner January 21, 2026 16:11
@DanielMSchmidt DanielMSchmidt added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Jan 21, 2026
@github-actions
Copy link
Contributor

The equivalence tests failed. Please investigate here.

@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch 2 times, most recently from f84a4ce to 6afd3af Compare January 21, 2026 16:33
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from bda61b0 to 280e593 Compare January 22, 2026 10:58
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from 6afd3af to de41238 Compare January 22, 2026 10:59
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from 280e593 to f5e79eb Compare January 22, 2026 12:24
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from de41238 to bdc972e Compare January 22, 2026 12:25
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from f5e79eb to 7e3ae80 Compare January 22, 2026 14:59
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch 5 times, most recently from d00ce77 to efac832 Compare January 23, 2026 12:45
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from 7e3ae80 to ced2884 Compare January 26, 2026 08:50
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from 9a3ecfc to 9ca210c Compare January 26, 2026 08:50
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from ced2884 to 1ae68b1 Compare January 27, 2026 13:05
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from 9ca210c to 1afde6d Compare January 27, 2026 13:05
jbardin
jbardin previously approved these changes Feb 2, 2026
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from 1ae68b1 to f06653d Compare February 3, 2026 12:27
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from 1afde6d to 6fd7c95 Compare February 3, 2026 12:30
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from c99ec3a to 03f545c Compare February 3, 2026 14:24
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from 6fd7c95 to c2e3016 Compare February 3, 2026 14:39
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from 03f545c to 73395dc Compare February 3, 2026 15:06
@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from c2e3016 to b5c6794 Compare February 3, 2026 15:09
@DanielMSchmidt DanielMSchmidt force-pushed the use-deprecation-marks-for-resource-deprecation-3 branch from 73395dc to dec3b8e Compare February 3, 2026 15:32
Base automatically changed from use-deprecation-marks-for-resource-deprecation-3 to main February 3, 2026 15:47
@DanielMSchmidt DanielMSchmidt dismissed jbardin’s stale review February 3, 2026 15:47

The base branch was changed.

@DanielMSchmidt DanielMSchmidt force-pushed the limit-usage-of-deep-unmarking-2 branch from b5c6794 to 0100adb Compare February 3, 2026 15:48
@DanielMSchmidt DanielMSchmidt merged commit ba5c4ac into main Feb 3, 2026
11 checks passed
@DanielMSchmidt DanielMSchmidt deleted the limit-usage-of-deep-unmarking-2 branch February 3, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants