Skip to content

MINOR: remove unused constructor param from ProcessorStateManager#2385

Closed
dguy wants to merge 1 commit intoapache:trunkfrom
dguy:minor-remove-unused-param
Closed

MINOR: remove unused constructor param from ProcessorStateManager#2385
dguy wants to merge 1 commit intoapache:trunkfrom
dguy:minor-remove-unused-param

Conversation

@dguy
Copy link
Copy Markdown
Contributor

@dguy dguy commented Jan 16, 2017

Remove applicationId parameter as it is no longer used.

@dguy
Copy link
Copy Markdown
Contributor Author

dguy commented Jan 16, 2017

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/911/
Test PASSed (JDK 8 and Scala 2.11).

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/909/
Test PASSed (JDK 8 and Scala 2.12).

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/909/
Test PASSed (JDK 7 and Scala 2.10).

@guozhangwang
Copy link
Copy Markdown
Contributor

Merged to trunk.

@asfgit asfgit closed this in b62804a Jan 16, 2017
asfgit pushed a commit that referenced this pull request Jan 17, 2017
Remove applicationId parameter as it is no longer used.

Author: Damian Guy <damian.guy@gmail.com>

Reviewers: Guozhang Wang <wangguoz@gmail.com>

Closes #2385 from dguy/minor-remove-unused-param
@dguy dguy deleted the minor-remove-unused-param branch January 19, 2017 11:40
soenkeliebau pushed a commit to soenkeliebau/kafka that referenced this pull request Feb 7, 2017
Remove applicationId parameter as it is no longer used.

Author: Damian Guy <damian.guy@gmail.com>

Reviewers: Guozhang Wang <wangguoz@gmail.com>

Closes apache#2385 from dguy/minor-remove-unused-param
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.

3 participants