Skip to content

MINOR: auto.offset.reset docs not in sync with validation#91

Closed
sslavic wants to merge 1 commit intoapache:trunkfrom
sslavic:patch-5
Closed

MINOR: auto.offset.reset docs not in sync with validation#91
sslavic wants to merge 1 commit intoapache:trunkfrom
sslavic:patch-5

Conversation

@sslavic
Copy link
Member

@sslavic sslavic commented Jul 21, 2015

In this commit 0699ff2#diff-5533ddc72176acd1c32f5abbe94aa672 among other things auto.offset.reset possible options were changed from smallest to earliest and from largest to latest, but not in documentation for that configuration property.

This patch fixes documentation for auto.offset.reset consumer configuration property so it is in sync with validation logic.

@asfbot
Copy link

asfbot commented Jul 21, 2015

kafka-trunk-git-pr #28 FAILURE
Looks like there's a problem with this pull request

@sslavic
Copy link
Member Author

sslavic commented Jul 21, 2015

Build is unstable, tests failing, but not due to this PR.

@hachikuji
Copy link
Contributor

+1

@ijuma
Copy link
Member

ijuma commented Jul 22, 2015

LGTM

@sslavic, not sure why but it seems like the flaky tests don't like your documentation updates. :) The tests have been passing for other PRs with actual code changes.

@sslavic
Copy link
Member Author

sslavic commented Jul 22, 2015

@ijuma thanks for reviewing my patches. Projects have different rules, what about Apache Kafka - is it a must to have a JIRA ticket for every commit? Am I supposed to create a JIRA ticket along with PR? Would it just speed up accepting of changes?

@ijuma
Copy link
Member

ijuma commented Jul 22, 2015

@sslavic The official documentation on contributing is here http://kafka.apache.org/contributing.html .
There is an ongoing vote in the mailing list to switch to GitHub pull requests for new contributions and that is described here https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes . It looks like the vote will pass and if that happens, then a JIRA ticket won't be required for trivial documentation changes such as this.

A separate question is how to speed up the process of integrating changes. Many of the current committers only monitor JIRA at this point since that's still the official way.

@sslavic sslavic changed the title auto.offset.reset docs not in sync with validation HOTFIX: auto.offset.reset docs not in sync with validation Jul 29, 2015
@sslavic sslavic changed the title HOTFIX: auto.offset.reset docs not in sync with validation MINOR: auto.offset.reset docs not in sync with validation Jul 29, 2015
In this commit apache@0699ff2#diff-5533ddc72176acd1c32f5abbe94aa672 among other things auto.offset.reset possible options were changed from smallest to earliest and from largest to latest, but not in documentation for that configuration property.

This patch fixes documentation for auto.offset.reset consumer configuration property so it is in sync with validation logic.
@asfbot
Copy link

asfbot commented Jul 30, 2015

kafka-trunk-git-pr #64 FAILURE
Looks like there's a problem with this pull request

@asfbot
Copy link

asfbot commented Jul 30, 2015

kafka-trunk-git-pr #66 FAILURE
Looks like there's a problem with this pull request

@asfgit asfgit closed this in 4b798f0 Aug 5, 2015
@guozhangwang
Copy link
Contributor

LGTM.

@sslavic sslavic deleted the patch-5 branch August 5, 2015 23:36
ymatsuda pushed a commit to ymatsuda/kafka that referenced this pull request Mar 2, 2016
…reams-tech-preview

Backport recent changes from trunk/streams: batch apache#2
rustd pushed a commit to rustd/pranavfinaldemokafka that referenced this pull request Feb 9, 2024
…e#12796) (apache#91)

This patch adds a unit test for topic recreation with colliding characters (such as `.`). This was broken up until apache#12790. 

Reviewers: José Armando García Sancio <[email protected]>

Co-authored-by: Jason Gustafson <[email protected]>
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
Clean up localstack setup left. Local executions use MinIO now.
fvaleri pushed a commit to fvaleri/kafka that referenced this pull request Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants