Skip to content

KAFKA-2203: Getting Java8 to relax about javadoc and let our build pass#147

Closed
gwenshap wants to merge 1 commit intoapache:trunkfrom
gwenshap:KAFKA-2203
Closed

KAFKA-2203: Getting Java8 to relax about javadoc and let our build pass#147
gwenshap wants to merge 1 commit intoapache:trunkfrom
gwenshap:KAFKA-2203

Conversation

@gwenshap
Copy link
Copy Markdown
Contributor

This patch is different than the one attached to the JIRA - I'm applying the new javadoc rules to all subprojects while the one in the JIRA applies only to "clients". We need this since Copycat has the same issues.

@gwenshap
Copy link
Copy Markdown
Contributor Author

Note that without this patch, "./gradlew install" fails with Java8

@ijuma
Copy link
Copy Markdown
Member

ijuma commented Aug 18, 2015

LGTM, ./gradlew clean install failed with Java 8 before the patch and now succeeds.

@asfbot
Copy link
Copy Markdown

asfbot commented Aug 18, 2015

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

@gwenshap
Copy link
Copy Markdown
Contributor Author

The test failure seems unrelated.

@guozhangwang, @junrao - I need a commit :)

@guozhangwang
Copy link
Copy Markdown
Contributor

LGTM, thanks.

@asfgit asfgit closed this in 7915396 Oct 12, 2015
apurvam pushed a commit to apurvam/kafka that referenced this pull request Apr 3, 2017
* handling add offsets to txn
* add a pending state with prepareTransition / completeTransaction / abortTransition of state
* refactor handling logic for multiple in-flight requests
dguy pushed a commit to dguy/kafka that referenced this pull request Apr 13, 2017
* handling add offsets to txn
* add a pending state with prepareTransition / completeTransaction / abortTransition of state
* refactor handling logic for multiple in-flight requests
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
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.

4 participants