search

confluentinc / camus

Mirror of Linkedin's Camus
Apache License 2.0
0 stars 62 forks source link

CC-1350: Temporarily corrected AK core’s missing dependency on SLF4J API #63

Closed rhauch closed 6 years ago

rhauch commented 7 years ago

Recent changes in the 1.1.0-SNAPSHOT version of AK added code that depends on the SLF4J API library, but AK is not currently including that in the org.apache:kafka_2.11 dependencies (see KAFKA-6313).

This commit temporarily adds them via the org.apache:kafka-clients library, which does depend on the correct version of SLF4J API. This corrects the problem, but we’ll want to revert this when KAFKA-6313 is fixed.

rhauch commented 7 years ago

@ewencp We have two options:

  1. Merge this now so that we fix the Camus build, and revert once KAFKA-6313 is fixed in apache/kafka and confluent/kafka.
  2. Close this without merging and wait for KAFKA-6313 to be fixed in apache/kafka and confluent/kafka.

Any preferences?