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

[Doc] Remove Elasticsearch sink obsolete documentation page #14695

Closed
1 task done
vroyer opened this issue Mar 15, 2022 · 7 comments · Fixed by #17603
Closed
1 task done

[Doc] Remove Elasticsearch sink obsolete documentation page #14695

vroyer opened this issue Mar 15, 2022 · 7 comments · Fixed by #17603
Labels
doc-required Your PR changes impact docs and you will update later. lifecycle/stale Stale

Comments

@vroyer
Copy link
Contributor

vroyer commented Mar 15, 2022

What issue do you find in Pulsar docs?

This elasticsearch sink page is obsolete.
The right one is now this page.

What is your suggestion?

Remove the obsolete page https://pulsar.apache.org/docs/en/io-elasticsearch/

Do you have any references?

https://www.datastax.com/products/luna-streaming

Would you like to fix this issue?

Yes, remove the io-elasticsearch.md

Note

  • I have researched my question.
@Anonymitaet
Copy link
Member

@vroyer please do not delete io-elasticsearch.md because other places might reference that file.
Instead, you can update https://pulsar.apache.org/docs/en/io-elasticsearch/ with https://pulsar.apache.org/docs/en/io-elasticsearch-sink/.

@github-actions
Copy link

The issue had no activity for 30 days, mark with Stale label.

@github-actions
Copy link

The issue had no activity for 30 days, mark with Stale label.

@momo-jun
Copy link
Contributor

momo-jun commented Jun 16, 2022

I've checked both master and versioned docs, the io-elasticsearch.md file is not referenced now.

@eolivelli
Copy link
Contributor

@nicoloboschi can you help here please ?

@nicoloboschi
Copy link
Contributor

@momo-jun @Anonymitaet both https://pulsar.apache.org/docs/io-elasticsearch-sink/ and https://pulsar.apache.org/docs/io-elasticsearch/ are obsolete. We can remove them. It's better to have a not found page instead of very old page.

The official page is https://pulsar.apache.org/docs/next/io-elasticsearch-sink#configuration

@github-actions github-actions bot removed the Stale label Jun 17, 2022
@github-actions
Copy link

The issue had no activity for 30 days, mark with Stale label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required Your PR changes impact docs and you will update later. lifecycle/stale Stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants