You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/05/06 16:23:00 UTC
[jira] [Commented] (FLINK-8655) Add a default keyspace to
CassandraSink
[ https://issues.apache.org/jira/browse/FLINK-8655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16465212#comment-16465212 ]
ASF GitHub Bot commented on FLINK-8655:
---------------------------------------
Github user ctamisier commented on the issue:
https://github.com/apache/flink/pull/5538
This is also something I would need and I would like to see with you for an alternative fix.
What about using the `Configuration` that is provided in `RichFunction.open(Configuration parameters)` for the `CassandraSinkBase.open(Configuration configuration) {...}` implementation ?
I saw in [old docs](https://ci.apache.org/projects/flink/flink-docs-release-1.3/dev/best_practices.html#passing-parameters-as-a-configuration-object-to-single-functions) that `Configuration` can be used in the `open(...)` method but today (1.4+) it might not be a good practice anymore.
What about adding `keyspace` attribute in `CassandraPojoSink` and `CassandraSinkBuilder` (throwing exception when not using a `CassandraPojoSinkBuilder` for the moment).
And create a new `Configuration()` with this `keyspace` in `CassandraPojoSink`.
And finally do a `cluster.connect(keyspace);`
I've done this [here](https://github.com/apache/flink/compare/master...ctamisier:pojo-keyspace) if you can have a look.
I've updated `CassandraConnectorITCase` with a new test.
I would like to run `CassandraPojoSinkExample.main()` to cover the `CassandraSink.addSink()` mechanism, but it doesn't work for me (even on flink master branch).
Can this be a candidate for a PR, I am new to flink so it might break the flink good practice principles...
Let me know!
> Add a default keyspace to CassandraSink
> ---------------------------------------
>
> Key: FLINK-8655
> URL: https://issues.apache.org/jira/browse/FLINK-8655
> Project: Flink
> Issue Type: Improvement
> Components: Cassandra Connector
> Affects Versions: 1.4.0
> Reporter: Christopher Hughes
> Priority: Minor
> Labels: features
> Fix For: 1.6.0
>
>
> Currently, to use the CassandraPojoSink, it is necessary for a user to provide keyspace information on the desired POJOs using datastax annotations. This allows various POJOs to be written to multiple keyspaces while sinking messages, but prevent runtime flexibility.
> For many developers, non-production environments may all share a single Cassandra instance differentiated by keyspace names. I propose adding a `defaultKeyspace(String keyspace)` to the ClusterBuilder. POJOs lacking a definitive keyspace would attempt to be loaded to the provided default.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)