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

HOTFIX: S3SinkConnector should extend SinkConnector #45

Conversation

kkonstantine
Copy link
Member

@kkonstantine kkonstantine commented May 8, 2017

  • Needed by the framework for better management of the connector.
  • Fixes CC-537: S3SinkConnector needs to extend SinkConnector for runtime class resolution

@kkonstantine kkonstantine requested review from ewencp and rhauch May 8, 2017 21:40
@norwood
Copy link
Contributor

norwood commented May 8, 2017

😍

@kkonstantine kkonstantine force-pushed the HOTFIX-S3SinkConnector-should-extend-SinkConnector-abstract-class branch from 25cbab4 to c65d04f Compare May 9, 2017 05:48
Copy link
Contributor

@ewencp ewencp left a comment

Choose a reason for hiding this comment

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

LGTM, good find!

  * Needed by the framework for better management of the connector.
  * Fixes CC-537: S3SinkConnector needs to extend SinkConnector for runtime class resolution
@kkonstantine kkonstantine force-pushed the HOTFIX-S3SinkConnector-should-extend-SinkConnector-abstract-class branch from c65d04f to 4142c1b Compare May 13, 2017 19:27
@kkonstantine kkonstantine merged commit 4142c1b into confluentinc:3.2.x May 13, 2017
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.

3 participants