[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-11 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73938104 @koeninger this is a definite blocker for me, I'm upgrading the connector to scala 2.11 with a cross build. Let me know if you have time, otherwise I will get back to

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-11 Thread koeninger
Github user koeninger commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73959340 @helena I updated it, pr is at https://github.com/apache/spark/pull/4537 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-11 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73979757 Great I'll close this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-11 Thread helena
Github user helena closed the pull request at: https://github.com/apache/spark/pull/3631 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-10 Thread koeninger
Github user koeninger commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73728226 This will need some changes to KafkaCluster and possibly other things related to the new api... let me know if you want a hand. --- If your project is set up for it,

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-10 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73728845 @koeninger Yes, there consumer offset and some other minor changes to make. I'm finding it hard to carve out time for this, high ticket load this week. Go for it if

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73368921 @helena I too would like to update to Kafka 0.8.2 as it does have a number of good improvements. can you rebase? @tdas are you OK with merging this for only master /

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-08 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73377050 Hi @srowen, will do. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73100513 Then for the new API (which does not store offsets externally) may need to be updated to ensure compatibility. I am not sure whether the current unit tests will catch such

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73100788 [Test build #26849 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26849/consoleFull) for PR 3631 at commit

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73100148 Jenkins, this is ok to test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73101317 NP, I think manual may not be necessary, just the addition of tests that automate testing the version or testing all and handling the execution path properly. I can add

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73120496 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-73120485 [Test build #26849 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26849/consoleFull) for PR 3631 at commit

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-04 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-72903283 The first step would be to see whether upgrading to 0.8.2 passes the inbuilt unit-tests or not (including the new Kafka stuff). In addition, is there are

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-04 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-72926976 All streaming-kafka sbt tests and /dev/run-tests passed in my initial push as beta, and with the GA update all still pass. Should I resubmit this PR against another

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-04 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-72858529 @tdas what shall I do with this PR to complete it then? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-03 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-72784310 Aah cool. However 0.8.1 and 0.8.2 have pretty big changes between them, so lets merge this for the next release. We are already doing a lot of experimental Kafka stuff in

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-02-03 Thread rustyrazorblade
Github user rustyrazorblade commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-72775510 Kafka 0.8.2 is now listed as latest stable. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2015-01-08 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-69206347 Waiting for kafka 0.8.2 to move to GA, still beta --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2014-12-23 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-68013540 Hey @helena I am not sure whether we want to upgrade to a beta version. Especially, there have been subtle changes within Kafka between releases and I am afraid to make

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2014-12-23 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-68013974 Hi @tdas, +1. This PR was done as initial push, and as soon as 0.8.2 is out of beta I planned to do the final commit, then have it considered. I never expected it to be

[GitHub] spark pull request: #SPARK-2808 update kafka to version 0.8.2

2014-12-22 Thread helena
Github user helena commented on the pull request: https://github.com/apache/spark/pull/3631#issuecomment-67836412 @JoshRosen Ticket name updated :) Sorry for the delay, I was away. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub