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

Mark cardinality limits as Stable #4222

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

reyang
Copy link
Member

@reyang reyang commented Sep 23, 2024

Mark cardinality limits as Stable.

Fixes #3904

These are the language implementations:

Changes

Please provide a brief description of the changes here.

For non-trivial changes, follow the change proposal process.

pellared
pellared previously approved these changes Sep 24, 2024
@cijothomas
Copy link
Member

There are few issues that were raised earlier on this topic. We should mark them as resolved (and close the issue), or mark them as non-blocking for stability for cardinality limits.
#3904 (comment)

@reyang
Copy link
Member Author

reyang commented Sep 25, 2024

... or mark them as non-blocking for stability for cardinality limits

How do you want to mark them as non-blocking? @cijothomas

@MrAlias
Copy link
Contributor

MrAlias commented Sep 25, 2024

I feel this issue needs a resolution before this can become stable.

@cijothomas
Copy link
Member

... or mark them as non-blocking for stability for cardinality limits

How do you want to mark them as non-blocking? @cijothomas

TC can leave a comment on the issues saying "it is not a blocker for stabilizing cardinality limits feature spec".
(I don't think it worth creating github labels like blocking-cardinality-stable, allowed-cardinality-stable, nice-to-have-cardinality-stable etc, so a simple comment can help.)

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.

Stabilize Overflow attribute section under Cardinality Limits
6 participants