[GitHub] incubator-streams pull request #345: [STREAMS-478] streams persist reader/wr...

2016-12-28 Thread sbcd90
Github user sbcd90 commented on a diff in the pull request: https://github.com/apache/incubator-streams/pull/345#discussion_r94092806 --- Diff: streams-contrib/streams-persist-cassandra/src/main/java/org/apache/streams/cassandra/CassandraPersistReader.java --- @@ -0,0 +1,316

[GitHub] incubator-streams issue #345: [STREAMS-478] streams persist reader/writer fo...

2016-12-28 Thread sbcd90
Github user sbcd90 commented on the issue: https://github.com/apache/incubator-streams/pull/345 Hi @smarthi , I have refactored the code to remove the `printStackTrace` method occurrence by adding `LOGGER.warn` statements. --- If your project is set up for it, you can reply

[GitHub] incubator-streams pull request #345: [STREAMS-478] streams persist reader/wr...

2016-12-24 Thread sbcd90
GitHub user sbcd90 opened a pull request: https://github.com/apache/incubator-streams/pull/345 [STREAMS-478] streams persist reader/writer for Apache Cassandra This issue is created to propose the streams persist reader/writer to get/post documents from/to apache cassandra. You

[GitHub] incubator-streams issue #345: [STREAMS-478] streams persist reader/writer fo...

2016-12-28 Thread sbcd90
Github user sbcd90 commented on the issue: https://github.com/apache/incubator-streams/pull/345 Hi @smarthi , @steveblackmon , Thanks for the review. I have refactored the code to handle the following issues: - removing of guava APIs. - no types specified