Skip to content

[SPARK-55300][DOCS] Add timestamp offset option visualization in Kafka docs#54080

Closed
kavpreetgrewal wants to merge 1 commit intoapache:masterfrom
kavpreetgrewal:update-kafka-offset-doc
Closed

[SPARK-55300][DOCS] Add timestamp offset option visualization in Kafka docs#54080
kavpreetgrewal wants to merge 1 commit intoapache:masterfrom
kavpreetgrewal:update-kafka-offset-doc

Conversation

@kavpreetgrewal
Copy link
Contributor

@kavpreetgrewal kavpreetgrewal commented Jan 31, 2026

What changes were proposed in this pull request?

Adds a diagram illustrating the behavior the the timestamp offset options for Kafka.

Why are the changes needed?

The text explanation of the timestamp offset options in the guide doc was not sufficient for many users to form the above illustration in their minds. Their feedback is something like “A picture is worth a thousand words”.

Does this PR introduce any user-facing change?

No. Documentation updates only.

How was this patch tested?

N/A. Verified that the docs build as expected.

Screenshot 2026-01-30 at 4 24 13 PM

site.zip

  • See /streaming/structured-streaming-kafka-integration.html

Was this patch authored or co-authored using generative AI tooling?

No. AI was only used for consultation purposes.

@github-actions github-actions bot added the DOCS label Jan 31, 2026
@github-actions
Copy link

github-actions bot commented Jan 31, 2026

JIRA Issue Information

=== Improvement SPARK-55300 ===
Summary: Visualize the behavior of startingOffset/endingOffset option in Kafka connector
Assignee: None
Status: Open
Affected: ["4.2.0"]


This comment was automatically generated by GitHub Actions

@HyukjinKwon HyukjinKwon changed the title [SPARK-55300][Docs] Add timestamp offset option visualization in Kafka docs [SPARK-55300][DOCS] Add timestamp offset option visualization in Kafka docs Feb 5, 2026
Copy link
Contributor

@HeartSaVioR HeartSaVioR left a comment

Choose a reason for hiding this comment

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

+1

Looks like CI failed but this is a pure doc change which should be irrelevant.

@HeartSaVioR
Copy link
Contributor

Thanks! Merging to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants