Skip to content

docs: Update tutorials to not use bin/post-index-task#19637

Open
avaamsel wants to merge 11 commits into
apache:masterfrom
avaamsel:update-post-index-task
Open

docs: Update tutorials to not use bin/post-index-task#19637
avaamsel wants to merge 11 commits into
apache:masterfrom
avaamsel:update-post-index-task

Conversation

@avaamsel

@avaamsel avaamsel commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Updated tutorials to no longer user bin/post-index-task and instead use cURL so users can directly call the Overlord API themselves.

Edited:

  • docs/ingestion/native-batch.md
  • docs/tutorials/tutorial-batch.md
  • docs/tutorials/tutorial-compaction.md
  • docs/tutorials/tutorial-delete-data.md
  • docs/tutorials/tutorial-ingestion-spec.md
  • docs/tutorials/tutorial-retention.md

This PR has:

  • been self-reviewed.
  • been tested in a test Druid cluster.

Comment thread docs/tutorials/tutorial-ingestion-spec.md Outdated
Comment thread docs/tutorials/tutorial-ingestion-spec.md Outdated
@avaamsel avaamsel marked this pull request as ready for review July 6, 2026 19:15

@avaamsel avaamsel left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We confirmed that in the file in tutorial-ingestion-spec was created earlier on in the tutorial and removed the duplicated content in tutorial-batch with the removal of the script from the file.

@317brian 317brian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for making these changes!

@FrankChen021 FrankChen021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Severity Findings
P0 0
P1 0
P2 2
P3 0
Total 2

This is an automated review by Codex GPT-5.5

Comment thread docs/tutorials/tutorial-batch.md
Comment thread docs/tutorials/tutorial-ingestion-spec.md Outdated

@FrankChen021 FrankChen021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Severity Findings
P0 0
P1 0
P2 2
P3 0
Total 2
Severity Findings
P0 0
P1 0
P2 2
P3 0
Total 2

Reviewed 6 of 6 changed files.

Found 2 tutorial correctness issues after replacing bin/post-index-task with raw curl.


This is an automated review by Codex GPT-5.5

Comment thread docs/tutorials/tutorial-batch.md
Comment thread docs/tutorials/tutorial-ingestion-spec.md Outdated

@FrankChen021 FrankChen021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The latest update fixes the prior tutorial command/path issues, and I found no new high-confidence PR-caused issues.

Reviewed 6 of 6 changed files.


This is an automated review by Codex GPT-5.5

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.

3 participants