Skip to content

Reduce visibility of Tranquility documentation#11134

Merged
sthetland merged 3 commits intoapache:masterfrom
sthetland:reduce-tranquility-visibility
May 3, 2021
Merged

Reduce visibility of Tranquility documentation#11134
sthetland merged 3 commits intoapache:masterfrom
sthetland:reduce-tranquility-visibility

Conversation

@sthetland
Copy link
Copy Markdown

@sthetland sthetland commented Apr 19, 2021

Since at least version 0.16.0-incubating we've documented that Druid no longer maintains compatibility with Tranquility.

We should accordingly reduce visibility of Tranquility in the documentation, removing it from the left nav and removing it as one of the streaming ingestion options from the table on the Ingestion overview page.

This PR takes those steps.

Not addressed in this PR, but a potential future step could be to remove the Tranquility page entirely, redirecting to a "Deprecated features" page, where it may be listed.

Also not addressed is at least one reference to Tranquility associated with the index_realtime task type. All other references to Tranquility mention that it should not be used for new projects.

@sthetland sthetland changed the title demote tranquility doc Reduce visibility of Tranquility documentation Apr 19, 2021
@sthetland sthetland marked this pull request as ready for review April 22, 2021 23:58
Copy link
Copy Markdown
Contributor

@techdocsmith techdocsmith left a comment

Choose a reason for hiding this comment

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

Minor nit. LGTM

Comment thread docs/ingestion/index.md Outdated
Co-authored-by: Charles Smith <38529548+techdocsmith@users.noreply.github.com>
@sthetland sthetland merged commit ca1412d into apache:master May 3, 2021
@sthetland sthetland deleted the reduce-tranquility-visibility branch May 3, 2021 23:48
@clintropolis clintropolis added this to the 0.22.0 milestone Aug 12, 2021
gianm added a commit to gianm/druid that referenced this pull request Jan 18, 2024
index_realtime tasks were removed from the documentation in apache#13107. Even
at that time, they weren't really documented per se— just mentioned. They
existed solely to support Tranquility, which is an obsolete ingestion
method that predates migration of Druid to ASF and is no longer being
maintained. Tranquility docs were also de-linked from the sidebars and
the other doc pages in apache#11134. Only a stub remains, so people with
links to the page can see that it's no longer recommended.

index_realtime_appenderator tasks existed in the code base, but were
never documented, nor as far as I am aware were they used for any purpose.

This patch removes both task types completely, as well as removes all
supporting code that was otherwise unused. It also updates the stub
doc for Tranquility to be firmer that it is not compatible. (Previously,
the stub doc said it wasn't recommended, and pointed out that it is
built against an ancient 0.9.2 version of Druid.)
clintropolis added a commit that referenced this pull request Jun 25, 2024
index_realtime tasks were removed from the documentation in #13107. Even
at that time, they weren't really documented per se— just mentioned. They
existed solely to support Tranquility, which is an obsolete ingestion
method that predates migration of Druid to ASF and is no longer being
maintained. Tranquility docs were also de-linked from the sidebars and
the other doc pages in #11134. Only a stub remains, so people with
links to the page can see that it's no longer recommended.

index_realtime_appenderator tasks existed in the code base, but were
never documented, nor as far as I am aware were they used for any purpose.

This patch removes both task types completely, as well as removes all
supporting code that was otherwise unused. It also updates the stub
doc for Tranquility to be firmer that it is not compatible. (Previously,
the stub doc said it wasn't recommended, and pointed out that it is
built against an ancient 0.9.2 version of Druid.)

ITUnionQueryTest has been migrated to the new integration tests framework and updated to use Kafka ingestion.

Co-authored-by: Gian Merlino <gianmerlino@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants