Skip to content
This repository has been archived by the owner on Apr 4, 2019. It is now read-only.

Merging kafka-connect-hdfs 3.3.0 to support Kafka 0.11 #49

Open
wants to merge 105 commits into
base: master
Choose a base branch
from

Conversation

lewisdawson
Copy link

This allows the connector to work with Kafka 0.11. I also upgraded Guava to 20.0 to fix the NoSuchMethodError I was seeing under #48.

Sriram Subramanian and others added 30 commits October 3, 2016 16:58
Fix issue with topic names containing dots, not being compatible with hive (replace dot with underscore)
After initial feedback/code review:
Added test
Removed unused imports accidentally added.
Issue 136 - Support topic with dots in hive.
* Update Readme

* Update Readme
  - Automatic commit via:
    tbr apply-feature-freeze -e prod --no-dry-run PLATFORM
  - Automatic commit via:
    tbr apply-feature-freeze -e prod --no-dry-run PLATFORM
  - Automatic commit via:
    tbr apply-feature-freeze -e prod --no-dry-run PLATFORM
* Handle primitive types in AvroRecordWriterProvider.

* Use named variable in test instead of repeating constants.
mageshn and others added 27 commits June 7, 2017 11:29
  - Automatic commit via:
    tbr apply-rc-tag --rc 2 --confluent-patch 2 --build temp-rc-settings -e prod PLATFORM --skip-projects security,confluent-cli --no-dry-run
  - Automatic commit via:
    tbr apply-rc-tag --rc 4 --confluent-patch 1 --build release -e prod CAMUS PLATFORM --no-dry-run
@PraveenSeluka
Copy link
Contributor

Hi @lewisdawson thanks for your work on this. Will love to discuss how we can test this and get it merged. Are you open for a chat ? (My email is praveen.seluka@gmail.com)

#!/usr/bin/env groovy
common {
slackChannel = '#connect-eng'
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the Jenkinsfile needed?

@OneCricketeer
Copy link

@lewisdawson Any plans to upgrade this PR to 4.0?

HDFS connect was refactored to use kafka-connect-storage-common

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.