[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35784695 --- Diff: core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala --- @@ -23,6 +23,8 @@ import javax.annotation.Nullable import

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126021604 Thanks @JDrit ! I will fix those final details as I merge. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35784052 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35784682 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35796975 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126088128 Re: the avro dependency, this is a net new dependency for core. Previously this came in via the hive module (the metastore dependency to be specific). I suppose it

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126112201 Okay sounds good. Thanks for looking at it Sean. - Patrick On Wed, Jul 29, 2015 at 1:37 PM, Sean Owen notificati...@github.com wrote: Re: the

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126040715 sorry I am trying to understand this serialization thing a bit better ... something doesn't make sense to me, but mostly outside of these changes. Once I get a handle

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126063730 Did @srowen look at the build change? Sean or I should be signing off on any dependency changes in the build. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126084533 @pwendell @JoshRosen ah sorry, I thought @vanzin @zsxwing gave it the thumbs up earlier -- just following the comments I forgot to get the approval of sean or you as

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126065124 There was a lot of up-thread discussion regarding the dependency change; my first comment on this PR was asking whether this introduced a new dependency, etc.

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-126060583 merged to master, thanks @JDrit --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7004 --- 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-746][CORE] Added Avro Serialization to ...

2015-07-27 Thread JDrit
Github user JDrit commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-125257800 @squito @JoshRosen Have I addressed all your issues, or is there anything else you would like me to do? --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-27 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-125279434 I'm going to defer to other reviewers who have been following this patch more closely. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-124537897 [Test build #1196 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1196/consoleFull) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-124573048 [Test build #1196 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1196/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35261143 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35266023 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123875296 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123875261 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123940828 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123940810 [Test build #38137 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38137/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123956225 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123956172 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123955693 Jenkins, retest this please --- 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-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123958303 [Test build #38157 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38157/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123958311 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35262959 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -161,6 +162,25 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35261569 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123857175 2 super minor comments, but otherwise lgtm! Also as a bit general house-keeping, can you put some of the benchmarks you have on the jira (since that serves as a

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35262721 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35263041 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -161,6 +162,25 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35263138 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35263784 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35265166 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35263272 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35263954 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35265438 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35265806 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -161,6 +162,25 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with Logging {

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r35265695 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -161,6 +162,25 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with Logging {

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123909575 [Test build #38114 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38114/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123909597 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123919007 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123918991 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-22 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-123918916 Jenkins, retest this please --- 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-746][CORE] Added Avro Serialization to ...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-122025940 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-122025951 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-122059098 [Test build #37523 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37523/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-122059291 Merged build finished. Test PASSed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34723675 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,151 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121745318 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121745295 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121743717 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121743738 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121751252 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121751318 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121750826 Jenkins, retest this please --- 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-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121749529 [Test build #37405 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37405/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121749603 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121774720 [Test build #37408 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37408/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121774824 Merged build finished. Test PASSed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121777631 **[Test build #37404 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37404/console)** for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-12135 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121789774 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121789744 [Test build #37417 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37417/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121764074 Merged build finished. Test FAILed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121767183 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121767164 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121760803 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121760780 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581021 --- Diff: core/pom.xml --- @@ -35,6 +35,16 @@ urlhttp://spark.apache.org//url dependencies dependency +

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581606 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581486 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34583682 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34584076 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34584417 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34585605 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -160,6 +162,21 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with Logging

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34585666 --- Diff: core/src/test/scala/org/apache/spark/serializer/GenericAvroSerializerSuite.scala --- @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581426 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121294001 Is it important to use ZLIB as opposed to one of the other compression libraries we already have? Is that how we get better compression? Its just a bit of a burden for

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581493 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34581825 --- Diff: core/src/main/scala/org/apache/spark/serializer/GenericAvroSerializer.scala --- @@ -0,0 +1,161 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7004#discussion_r34584579 --- Diff: core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala --- @@ -72,6 +74,9 @@ class KryoSerializer(conf: SparkConf) private val

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121311564 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121311604 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121363467 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121363416 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121349416 [Test build #37238 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37238/console) for PR 7004 at commit

[GitHub] spark pull request: [SPARK-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121349556 Merged build finished. Test PASSed. --- 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-746][CORE] Added Avro Serialization to ...

2015-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7004#issuecomment-121367942 Merged build finished. Test FAILed. --- 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

  1   2   >