search

Aiven-Open / tiered-storage-for-apache-kafka

RemoteStorageManager for Apache Kafka® Tiered Storage
Apache License 2.0
95 stars 20 forks source link

Add GCS storage implementation #361

Closed ivanyu closed 1 year ago

ivanyu commented 1 year ago

See individual commits for details.

jeqo commented 1 year ago

Looks dependency checker is working :) A couple of dependencies seem to conflict. Wondering if we should provide a tar per storage back-end if the conflict is between S3 and GCS for instance 🤔

ivanyu commented 1 year ago

Looks dependency checker is working :) A couple of dependencies seem to conflict. Wondering if we should provide a tar per storage back-end if the conflict is between S3 and GCS for instance 🤔

The conflict is between GCS and Checkstyle. For some reason, Checkstyle stuff is in the runtime class path. We probably need to exclude Checkstyle altogether from this check, however I couldn't quickly find how.

ivanyu commented 1 year ago

@jeqo I'm planning to exclude validateDependencies from the check dependencies. I can't quickly find how to make it work with Checkstyle. We can fix it later.