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.

Sign up for GitHub

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

Jump to bottom

[FLINK-4238] Only allow/require query for Tuple Stream in CassandraSink #2273

Closed
wants to merge 2 commits into from
Closed

[FLINK-4238] Only allow/require query for Tuple Stream in CassandraSink #2273

wants to merge 2 commits into from

Conversation

aljoscha
Copy link
Contributor

R: @zentol for review

@zentol
Copy link
Contributor

zentol commented Jul 20, 2016

Since we now have separate branches for both sink types it may be worthwhile to move these checks completely into the respective CassandraXXXSinkBuilder classes. Otherwise, +1.

@aljoscha
Copy link
Contributor Author

Thanks for review! I moved the checks to the specific builders.

@aljoscha aljoscha closed this Jul 21, 2016
@aljoscha aljoscha deleted the cassandra-tuple-query branch July 21, 2016 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
component=Connectors/Cassandra
Projects
None yet
Milestone
No milestone
3 participants
@aljoscha @zentol @rmetzger