{"action":"created","issue":{"url":"https://api.github.com/repos/apache/flink/issues/2633","repository_url":"https://api.github.com/repos/apache/flink","labels_url":"https://api.github.com/repos/apache/flink/issues/2633/labels{/name}","comments_url":"https://api.github.com/repos/apache/flink/issues/2633/comments","events_url":"https://api.github.com/repos/apache/flink/issues/2633/events","html_url":"https://github.com/apache/flink/pull/2633","id":183033322,"number":2633,"title":"[FLINK-4497] CassadraSink support for scala tuples/case classes","user":{"login":"zentol","id":5725237,"avatar_url":"https://avatars.githubusercontent.com/u/5725237?v=3","gravatar_id":"","url":"https://api.github.com/users/zentol","html_url":"https://github.com/zentol","followers_url":"https://api.github.com/users/zentol/followers","following_url":"https://api.github.com/users/zentol/following{/other_user}","gists_url":"https://api.github.com/users/zentol/gists{/gist_id}","starred_url":"https://api.github.com/users/zentol/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zentol/subscriptions","organizations_url":"https://api.github.com/users/zentol/orgs","repos_url":"https://api.github.com/users/zentol/repos","events_url":"https://api.github.com/users/zentol/events{/privacy}","received_events_url":"https://api.github.com/users/zentol/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-10-14T12:04:28Z","updated_at":"2016-10-26T14:44:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/flink/pulls/2633","html_url":"https://github.com/apache/flink/pull/2633","diff_url":"https://github.com/apache/flink/pull/2633.diff","patch_url":"https://github.com/apache/flink/pull/2633.patch"},"body":"This PR adds support for scala tuples/case classes to the streaming at-least-once-sink. If this pattern works we can extend it to other sinks as well in a follow-up."},"comment":{"url":"https://api.github.com/repos/apache/flink/issues/comments/256369373","html_url":"https://github.com/apache/flink/pull/2633#issuecomment-256369373","issue_url":"https://api.github.com/repos/apache/flink/issues/2633","id":256369373,"user":{"login":"StephanEwen","id":1727146,"avatar_url":"https://avatars.githubusercontent.com/u/1727146?v=3","gravatar_id":"","url":"https://api.github.com/users/StephanEwen","html_url":"https://github.com/StephanEwen","followers_url":"https://api.github.com/users/StephanEwen/followers","following_url":"https://api.github.com/users/StephanEwen/following{/other_user}","gists_url":"https://api.github.com/users/StephanEwen/gists{/gist_id}","starred_url":"https://api.github.com/users/StephanEwen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StephanEwen/subscriptions","organizations_url":"https://api.github.com/users/StephanEwen/orgs","repos_url":"https://api.github.com/users/StephanEwen/repos","events_url":"https://api.github.com/users/StephanEwen/events{/privacy}","received_events_url":"https://api.github.com/users/StephanEwen/received_events","type":"User","site_admin":false},"created_at":"2016-10-26T14:44:27Z","updated_at":"2016-10-26T14:44:27Z","body":"Look pretty good.\r\n\r\nI think it would be good to try to make the Cassandra Module to the user Scala independent. That way, we don't introduce a hard requirement for Scala versioning and make this Scala version independent as soon as Flink's runtime is Scala Version independent.\r\n\r\nWe can do that the following way:\r\n\r\n  - Set the Scala dependency to \"provided\". Flink pulls Scala anyways, Scala users have Scala anyways always as a dependency. Users that use Scala from Java can always add an additional Scala dependency in their project.\r\n  - Add the Scala library as a test dependency\r\n  - Drop the Scala compiler plugin\r\n  - Use a case class for the tests that comes in the Scala library (such as Tuple2)\r\n  - Migrate the test to Java - the sink class is in Java anyways\r\n\r\nWhat do you think about that?\r\n"}}
{"actor":{"display_login":"StephanEwen"}}