Skip to content

Added azure-storage-blob-shaded.#217

Open
Alex Diachenko (avocader) wants to merge 4 commits intomasterfrom
azure-storage-blob-shaded
Open

Added azure-storage-blob-shaded.#217
Alex Diachenko (avocader) wants to merge 4 commits intomasterfrom
azure-storage-blob-shaded

Conversation

@avocader
Copy link
Contributor

@avocader Alex Diachenko (avocader) commented Nov 3, 2021

Problem

Azure Blob Storage Source Connector uses and ships with com.azure.azure-storage-blob dependency.
But, if the classpath has the older versions of this dependency - the connector will fail because of conflicting versions.

After bringing up the connector using the Docker playground, I see that Worker's classpath has 2 versions of this JAR:

[main] INFO org.apache.kafka.common.utils.AppInfoParser - Kafka startTimeMs: 1635903919086
  jvm.classpath = .../usr/share/java/confluent-security/connect/azure-storage-blob-12.5.0.jar:.../:/usr/share/java/kafka/azure-storage-blob-12.6.0.jar:.../usr/bin/../share/java/kafka/azure-storage-blob-12.6.0.jar:

Solution

Does this solution apply anywhere else?
  • yes
  • no
If yes, where?

Test Strategy

Testing done:
  • Unit tests
  • Integration tests
  • System tests
  • Manual tests

Release Plan

@avocader Alex Diachenko (avocader) requested a review from a team November 3, 2021 02:46
@avocader Alex Diachenko (avocader) marked this pull request as ready for review November 4, 2021 07:05
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.

1 participant