Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency management for Kafka is incomplete #16728

Closed
wants to merge 1 commit into from

Conversation

garyrussell
Copy link
Contributor

This jar is now a dependency of spring-kafka-test which is added in
test scope by initializr.

See spring-projects/spring-kafka#1084

This jar is now a dependency of spring-kafka-test which is added in
test scope by initializr.

See spring-projects/spring-kafka#1084
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 6, 2019
@philwebb philwebb added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels May 6, 2019
@philwebb philwebb added this to the 2.2.x milestone May 6, 2019
@snicoll
Copy link
Member

snicoll commented May 7, 2019

That feels a candidate to 2.1.x as well as one might argue that our dependency management for Kafka is missing a library.

@wilkinsona
Copy link
Member

+1 for adding this in 2.1.x. I think we should review the other entries and see what else is missing too.

@snicoll snicoll changed the title Add kafka-streams-test-utils to dependency Dependency management for Kafka is missing kafka-streams-test-utils May 7, 2019
@snicoll snicoll added type: bug A general bug and removed type: enhancement A general enhancement labels May 7, 2019
@snicoll snicoll modified the milestones: 2.2.x, 2.1.x May 7, 2019
@wilkinsona wilkinsona self-assigned this May 8, 2019
@wilkinsona wilkinsona changed the title Dependency management for Kafka is missing kafka-streams-test-utils Dependency management for Kafka is incomplete May 8, 2019
@wilkinsona
Copy link
Member

Minus a few examples projects, we're missing dependency management for the following modules:

  • connect-basic-auth-extension
  • kafka-streams-scala_2.11
  • kafka-streams-scala_2.12
  • kafka-streams-test-utils

@wilkinsona
Copy link
Member

Thank you, @garyrussell. The proposed addition to the dependency management is now in 2.1.x and master along with the other missing modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants