Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8276
---
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 enab
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134723995
Alright I'm merging this to master. Note that I'm not porting this
branch-1.5 as I think this is a relatively big change and it doesn't have any
immediate feature benef
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134722641
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134722643
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134722522
[Test build #41537 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41537/console)
for PR 8276 at commit
[`eae3341`](https://github.
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134686363
LGTM
---
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 an
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134673844
[Test build #41537 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41537/consoleFull)
for PR 8276 at commit
[`eae3341`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134672484
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134672520
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 have
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134671657
Thanks @sun-rui -- Change LGTM.
@davies Any other comments ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134671515
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134523227
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134523229
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134518738
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134518705
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134515607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134515605
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134515500
[Test build #41518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41518/console)
for PR 8276 at commit
[`0d82eae`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134477053
[Test build #41518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41518/consoleFull)
for PR 8276 at commit
[`0d82eae`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134476227
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134476212
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 h
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37832961
--- Diff: R/pkg/inst/tests/test_Serde.R ---
@@ -0,0 +1,67 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134472240
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134472241
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134472211
**[Test build #41504 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41504/console)**
for PR 8276 at commit
[`9025c9f`](https://github
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37829318
--- Diff: R/pkg/inst/tests/test_Serde.R ---
@@ -0,0 +1,67 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134443896
[Test build #41504 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41504/consoleFull)
for PR 8276 at commit
[`9025c9f`](https://gith
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134443211
The test passed on my machine, I don't know the reason. Anyway, add spark
context initialization into test_Serde to see if it can pass on Jenkins.
---
If your project i
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134443143
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134443161
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 have
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134413057
@sun-rui looks like the test failures are related to this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134404199
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134404198
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134404148
[Test build #41469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41469/console)
for PR 8276 at commit
[`fd0d086`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134360410
[Test build #41469 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41469/consoleFull)
for PR 8276 at commit
[`fd0d086`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134359836
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134359798
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 h
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134359309
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134353241
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134353239
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134353089
[Test build #41462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41462/console)
for PR 8276 at commit
[`fd0d086`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134306551
[Test build #41462 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41462/consoleFull)
for PR 8276 at commit
[`fd0d086`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134305750
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134305778
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 have
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134305683
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134304021
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134304025
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134303924
**[Test build #41453 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41453/console)**
for PR 8276 at commit
[`fd0d086`](https://github
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134219790
[Test build #41453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41453/consoleFull)
for PR 8276 at commit
[`fd0d086`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134217519
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134217584
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 have
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134217045
rebased to master
---
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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134121092
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134120923
[Test build #41446 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41446/console)
for PR 8276 at commit
[`3c56872`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134121090
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 project does n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134105748
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 project does n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134105749
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134105700
[Test build #41445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41445/console)
for PR 8276 at commit
[`9bfa62d`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134068634
[Test build #41446 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41446/consoleFull)
for PR 8276 at commit
[`3c56872`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134066574
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 have thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134066650
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 have this f
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37723615
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -98,27 +98,17 @@ private[r] object SQLUtils {
val bos = new ByteAr
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134066063
@shivaram , test cases for SerDe added. Now the SerDe does not support
transferring a list of different element types from R side to JVM side. Let's
leave it for future'
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134065726
[Test build #41445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41445/consoleFull)
for PR 8276 at commit
[`9bfa62d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134064728
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 have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134064672
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 have thi
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37501357
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -98,27 +98,17 @@ private[r] object SQLUtils {
val bos = new ByteA
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132900531
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132900528
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132900347
[Test build #41301 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41301/console)
for PR 8276 at commit
[`edc9652`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132869908
[Test build #41301 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41301/consoleFull)
for PR 8276 at commit
[`edc9652`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132868272
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132868301
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 have
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37490860
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -98,27 +98,20 @@ private[r] object SQLUtils {
val bos = new ByteAr
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37490835
--- Diff: R/pkg/R/DataFrame.R ---
@@ -628,18 +628,49 @@ setMethod("dim",
setMethod("collect",
signature(x = "DataFrame"),
fu
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37490241
--- Diff: R/pkg/R/DataFrame.R ---
@@ -628,18 +628,49 @@ setMethod("dim",
setMethod("collect",
signature(x = "DataFrame"),
fu
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132852075
will add test cases.
---
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 f
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132782565
I took a more detailed look at the code and I only had some minor comments
inline. So I think it looks pretty good but I think taking this opportunity to
add some tests
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37466121
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -98,27 +98,20 @@ private[r] object SQLUtils {
val bos = new ByteA
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37465795
--- Diff: R/pkg/R/DataFrame.R ---
@@ -628,18 +628,49 @@ setMethod("dim",
setMethod("collect",
signature(x = "DataFrame"),
f
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37465341
--- Diff: R/pkg/R/DataFrame.R ---
@@ -628,18 +628,49 @@ setMethod("dim",
setMethod("collect",
signature(x = "DataFrame"),
f
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132723164
Its fine. Lets not complicate this PR with the `listToSeq` thing as that
has its own issues like you mention.
I think just adding unit tests to SerDe is a good idea
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132571243
@shivaram, I tried to support ArrayType. By adding code like:
// Convert Seq[Any] to Array[Any]
val value =
if (obj.isInstanceOf[Seq[Any]])
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132505479
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132505478
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132505413
**[Test build #41220 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41220/console)**
for PR 8276 at commit
[`63def4c`](https://github
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132493158
I see. I think a separate PR for ArrayType is fine, its just harder to
review this change if we can't test it. Its ok, I will take one more closer
look tomorrow
---
I
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132478924
@shivaram, now ArrayType in a DataFrame is still not supported, as
ArrayType's class is something like
scala.collection.mutable.WrappedArray$ofRef, it will be passed as
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132461114
[Test build #41220 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41220/consoleFull)
for PR 8276 at commit
[`63def4c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132459146
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132459112
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 h
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132440650
BTW does this fix SPARK-9302 as well or does that require `struct` support ?
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132440124
[Test build #41188 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41188/console)
for PR 8276 at commit
[`7dea6fb`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132440142
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132440143
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132418166
@sun-rui We can do the `createDataFrame` in another PR, but for this PR can
we add a test case using a JSON file which has an `ArrayType` in it ?
---
If your project i
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132409487
[Test build #41188 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41188/consoleFull)
for PR 8276 at commit
[`7dea6fb`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132408494
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-132408482
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 h
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37371028
--- Diff: core/src/main/scala/org/apache/spark/api/r/SerDe.scala ---
@@ -210,22 +213,31 @@ private[spark] object SerDe {
writeType(dos, "void")
1 - 100 of 111 matches
Mail list logo