Skip to content

Conversation

@goguul
Copy link
Member

@goguul goguul commented Jan 28, 2026

No description provided.

- Moved dooming Release before dooming BaseAssets.
- Removed distinct as union already handles it.
- Removed `is not null` from the exclusion.
- Removed `is not null` condition for ValueStreamID as it cannot be null.
- doom ValueStreams ever used by doomed Releases,
except those ever used by safe Releases
Copy link
Contributor

@spazmodius spazmodius left a comment

Choose a reason for hiding this comment

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

LGTM

@lremedi
Copy link
Contributor

lremedi commented Jan 28, 2026

These fixes need to be replicated at https://github.com/versionone/IT-shared/tree/master/demo-data

@goguul goguul merged commit a375583 into master Jan 29, 2026
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.

4 participants