-
Notifications
You must be signed in to change notification settings - Fork 469
Remove "Enterprise" from v25.2 cdc docs and deprecate experimental changefeed syntax #19680
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
Conversation
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
7ea7580
to
3ecea93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM i love everything about this - THANK YOU :-)
@@ -41,8 +39,6 @@ Before you run the examples, verify that you have the `CHANGEFEED` privilege in | |||
|
|||
In this example, you'll set up a changefeed for a single-node cluster that is connected to a Kafka sink. The changefeed will watch two tables. | |||
|
|||
1. If you do not already have one, [request a trial {{ site.data.products.enterprise }} license]({% link {{ page.version.version }}/licensing-faqs.md %}#obtain-a-license). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YESSSSSSSSSS
toc: true | ||
docs_area: stream_data | ||
--- | ||
|
||
Core and {{ site.data.products.enterprise }} changefeeds offer different levels of configurability. {{ site.data.products.enterprise }} changefeeds allow for active changefeed jobs to be [paused](#pause), [resumed](#resume), and [canceled](#cancel). | ||
Changefeeds offer different levels of configurability. Changefeeds emitting to a sink allow for active changefeed jobs to be [paused](#pause), [resumed](#resume), and [canceled](#cancel). Sinkless changefeeds streams changes directly to the SQL session. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: double plural, i think this could read "Sinkless changefeeds stream changes ..."
@@ -53,46 +53,39 @@ For further detail on performance-related configuration, refer to the [Advanced | |||
- After you [restore from a full-cluster backup]({% link {{ page.version.version }}/restore.md %}#full-cluster), changefeed jobs will **not** resume on the new cluster. It is necessary to manually create the changefeeds following the full-cluster restore. | |||
- {% include {{ page.version.version }}/cdc/virtual-computed-column-cdc.md %} | |||
|
|||
The following Enterprise and Core sections outline how to create and configure each type of changefeed: | |||
The following sections outline how to create and configure each type of changefeed: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is why i no longer ever say a number of anything, only ever "the following things ..." 🤣
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YES, ever since you mentioned this to me in another review, I'm trying to remove those where I notice. And "either"
TFTRs! |
edb015f
to
2144a15
Compare
First PR to ensure "Enterprise" is removed from cdc docs — this is only for v25.2 (most of the changes are mechanical)
Deprecates
EXPERIMENTAL CHANGEFEED FOR
. PromotesCREATE CHANGEFEED
without sink syntax as our "sinkless" changefeed product option.Part of DOC-13882 (PR 1 of 3)