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

Broker features #720

Merged
merged 16 commits into from
Nov 7, 2023
Merged

Broker features #720

merged 16 commits into from
Nov 7, 2023

Conversation

tisonkun
Copy link
Member

@tisonkun tisonkun commented Oct 23, 2023

This PR refers to apache/pulsar#21128.

I commit the following months to work on this page and improve docs of features when we spot any. This page is now not referred from any of the other page and I plan to add some links of it in the docs portal when we have the first reasonable version of this matrix.

Preview
image

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@github-actions github-actions bot added the doc Improvements or additions to documentation label Oct 23, 2023
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@tisonkun
Copy link
Member Author

cc @jak78 @urfreespace @apache/pulsar-committers

data/broker-features.js Outdated Show resolved Hide resolved
feature: "Subscription / KeyShared",
experimentalSince: "",
betaSince: "",
stableSince: "",
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be 2.9.5

Copy link
Contributor

Choose a reason for hiding this comment

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

@codelipenghui It's better to use feature release version, not bug fix release?

Copy link
Member Author

Choose a reason for hiding this comment

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

@Jason918 I will use this patch release but I understand your concern that from a user point of view we need a x.0 stable version so that we're sure that all versions after that should be stable.

@codelipenghui for example, 2.9.5 is stable, but what if 2.10.0? I guess that the significant patch landed on 2.9.5 is not included in 2.10.0.

Anyway, we can use a pattern like 2.9.5/2.10.<n>/2.11.0 or later.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, it should be 2.10.4 for the Key_Shared subscription.

data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
data/broker-features.js Outdated Show resolved Hide resolved
@tisonkun
Copy link
Member Author

tisonkun commented Nov 7, 2023

Merging..

I'll send a notice to dev@ and seek a way to improve the matrix & expose it to other famous pages, e.g.:

  1. Doc portal
  2. Concept protal
  3. Selected feature pages (under constructing)

@tisonkun
Copy link
Member Author

tisonkun commented Nov 7, 2023

Thank @codelipenghui a lot for your help!

@tisonkun tisonkun merged commit 2748cff into apache:main Nov 7, 2023
2 checks passed
@tisonkun tisonkun deleted the broker-features branch November 7, 2023 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants