Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

[PDS-10{3841,4895}] Part 1: Revert Automated Stream Store Cleanup #4434

Merged
merged 5 commits into from
Nov 22, 2019

Conversation

jeremyk-91
Copy link
Contributor

Goals (and why):

Implementation Description (bullets):

  • Update generated code

Testing (What was existing testing like? What have you done to improve it?):

  • Updated stream test to verify that these streams are not cleaned up.

Concerns (what feedback would you like?):

  • Is the revert safe?
  • Are there edge cases in the stream store code I might have missed out on?

Where should we start reviewing?: StreamStoreRenderer

Priority (whenever / two weeks / yesterday): yesterday 🔥

@changelog-app
Copy link

changelog-app bot commented Nov 21, 2019

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Stream stores now no longer automatically clean up streams that were stored but never marked as used. We have noticed an increasing incidence of said cleaning being more aggressive than we anticipated.

Users that use stream stores must regenerate their schemas as part of upgrading to this version.

Check the box to generate changelog(s)

  • Generate changelog entry

@@ -94,15 +94,14 @@ public void targetedSweepLargeStreamsTest() {
}

@Test
public void targetedSweepCleanupUnmarkedStreamsTest() {
// TODO (jkong): This is obviously not the desired behaviour, but we are doing this in the interest of safety.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we revert this and mark this as @Ignore

@jeremyk-91 jeremyk-91 merged commit 2c2fc3c into develop Nov 22, 2019
@delete-merged-branch delete-merged-branch bot deleted the jkong/fire-1 branch November 22, 2019 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants