Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12640#discussion_r61540468
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/linalg/UDTSerializationBenchmark.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache S
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12640#issuecomment-215641970
**[Test build #57320 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57320/consoleFull)**
for PR 12640 at commit
[`537e363`](https://gi
Github user taku-k commented on the pull request:
https://github.com/apache/spark/pull/12767#issuecomment-215640922
cc @feynmanliang @jkbradley @mengxr
---
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
Github user rxin closed the pull request at:
https://github.com/apache/spark/pull/12722
---
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 enabl
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12722#issuecomment-215639874
Closing this one as https://github.com/apache/spark/pull/12770 subsumes
this.
---
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/12762#issuecomment-215639037
**[Test build #57319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57319/consoleFull)**
for PR 12762 at commit
[`fb443ef`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215638919
**[Test build #2932 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2932/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12762#discussion_r61539561
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala ---
@@ -58,6 +58,30 @@ object MLTestingUtils extends SparkFunSuite {
"
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12762#discussion_r61539464
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala ---
@@ -58,6 +58,30 @@ object MLTestingUtils extends SparkFunSuite {
"
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/12680#issuecomment-215638319
I'm glad to help.
---
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 user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12762#discussion_r61539432
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/recommendation/ALSSuite.scala ---
@@ -512,6 +513,60 @@ class ALSSuite
assert(getFactors(model.user
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12640#discussion_r61539393
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -336,4 +336,62 @@ public UnsafeArrayData copy()
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12759#issuecomment-215638073
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12759#issuecomment-215638072
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 projec
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12762#discussion_r61539327
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -53,24 +53,43 @@ import org.apache.spark.util.random.XORShiftRandom
*
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637950
**[Test build #57318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57318/consoleFull)**
for PR 12770 at commit
[`bf03065`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-215637933
cc @kayousterhout
---
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637923
**[Test build #2935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2935/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12759#issuecomment-215637965
**[Test build #57310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57310/consoleFull)**
for PR 12759 at commit
[`9b0d518`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637902
**[Test build #2934 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2934/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637873
**[Test build #2933 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2933/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-215637396
**[Test build #57317 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57317/consoleFull)**
for PR 12775 at commit
[`ee34cd2`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637280
**[Test build #2932 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2932/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637301
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637303
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215637293
**[Test build #57316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57316/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://g
GitHub user lirui-intel opened a pull request:
https://github.com/apache/spark/pull/12775
[SPARK-14958][Core] Failed task not handled when there's error
deserializing failure reason
## What changes were proposed in this pull request?
TaskResultGetter tries to deserialize th
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215635791
**[Test build #57316 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57316/consoleFull)**
for PR 12770 at commit
[`e68b453`](https://gi
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215635688
The current build dependency changes seem tentatively okay to me. Removing
the ASM 3 dependency should not be a problem because Spark depends on a shaded
copy anyways
Github user taku-k commented on a diff in the pull request:
https://github.com/apache/spark/pull/12767#discussion_r61536791
--- Diff: python/pyspark/ml/tests.py ---
@@ -586,10 +589,13 @@ def test_fit_maximize_metric(self):
tvsModel = tvs.fit(dataset)
bestMo
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12755
---
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 ena
Github user flyjy commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-215632760
@srowen The PR with unit testing passed after rebasing master
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/12755#issuecomment-215632671
Merging in master. Thanks!
---
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 t
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/12426#issuecomment-215632614
@mengxr - We should add details about this in SparkR programming guide. Can
you add this to the QA/docs JIRA we have for 2.0 ?
---
If your project is set up for it,
Github user taku-k commented on a diff in the pull request:
https://github.com/apache/spark/pull/12767#discussion_r61536588
--- Diff: python/pyspark/ml/tests.py ---
@@ -616,6 +622,7 @@ def test_save_load(self):
tvsModel.save(tvsModelPath)
loadedModel = Trai
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215632358
**[Test build #2931 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2931/consoleFull)**
for PR 12770 at commit
[`b74f5a5`](https://
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-215632322
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-215632323
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12765#issuecomment-215632276
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12765#issuecomment-215632274
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-215632229
**[Test build #57311 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57311/consoleFull)**
for PR 11812 at commit
[`bb03e08`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12765#issuecomment-215632186
**[Test build #57306 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57306/consoleFull)**
for PR 12765 at commit
[`923b92a`](https://g
Github user taku-k commented on a diff in the pull request:
https://github.com/apache/spark/pull/12767#discussion_r61536420
--- Diff: python/pyspark/ml/tuning.py ---
@@ -613,7 +615,9 @@ def copy(self, extra=None):
"""
if extra is None:
extr
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12755#issuecomment-215632083
**[Test build #2930 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2930/consoleFull)**
for PR 12755 at commit
[`ef6c1fb`](https://
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12596
---
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 ena
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12680#issuecomment-215631757
@GayathriMurali Since the feature freeze deadline is coming and there are
some follow-up tasks blocked by this PR and #12683, do you mind @yanboliang
sending out new PRs
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12426
---
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 ena
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/12596#issuecomment-215631602
LGTM
Merging with master
Thank you!
---
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 pr
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12426#issuecomment-215631563
LGTM2. Merged into master. Thanks!
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215630612
**[Test build #2931 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2931/consoleFull)**
for PR 12770 at commit
[`b74f5a5`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215630110
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215630103
**[Test build #57314 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57314/consoleFull)**
for PR 12764 at commit
[`234356a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215630111
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215630046
**[Test build #57315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57315/consoleFull)**
for PR 12764 at commit
[`fbfd06b`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215629908
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215629910
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215629896
**[Test build #57313 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57313/consoleFull)**
for PR 12770 at commit
[`b74f5a5`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215629614
**[Test build #57314 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57314/consoleFull)**
for PR 12764 at commit
[`234356a`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12771#issuecomment-215629419
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12771#issuecomment-215629417
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12771#issuecomment-215629333
**[Test build #57307 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57307/consoleFull)**
for PR 12771 at commit
[`461ab81`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215629230
**[Test build #57313 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57313/consoleFull)**
for PR 12770 at commit
[`b74f5a5`](https://gi
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12707#issuecomment-215628995
+1 on merging this first and fixing remaining issues later.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/12426#issuecomment-215628939
This looks pretty good to me. @mengxr @felixcheung any other comments ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12493#issuecomment-215628799
**[Test build #57312 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57312/consoleFull)**
for PR 12493 at commit
[`3efe9f5`](https://gi
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/12493#issuecomment-215628604
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12738#issuecomment-215628414
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215628448
**[Test build #2926 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2926/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12759#issuecomment-215628417
**[Test build #57310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57310/consoleFull)**
for PR 12759 at commit
[`9b0d518`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12738#issuecomment-215628413
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-215628418
**[Test build #57311 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57311/consoleFull)**
for PR 11812 at commit
[`bb03e08`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12738#issuecomment-215628390
**[Test build #57308 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57308/consoleFull)**
for PR 12738 at commit
[`161aa91`](https://g
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/12759#issuecomment-215628056
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 this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215627923
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215627896
**[Test build #57309 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57309/consoleFull)**
for PR 12770 at commit
[`2a84533`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215627926
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12774#issuecomment-215627856
(I think "(If this patch involves UI changes, please attach a screenshot;
otherwise, remove this)" can be removed in the PR description)
---
If your project is set
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215627564
**[Test build #2928 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2928/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215627327
**[Test build #2927 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2927/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12769#discussion_r61534645
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -132,7 +132,6 @@ statement
hiveNativeCommands
--- End d
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12769#discussion_r61534626
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -132,7 +132,6 @@ statement
hiveNativeCommands
--- End d
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/12774#discussion_r61534612
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/fileSourceInterfaces.scala
---
@@ -376,14 +376,10 @@ class HDFSFileCatalog(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12769
---
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 ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12773
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12769#issuecomment-215627044
Merging in 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 rxin commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215626966
I'm going to merge this. Let's see how the tests pan out.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12769#issuecomment-215626945
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12769#issuecomment-215626944
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215627013
**[Test build #2925 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2925/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215626991
**[Test build #2929 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2929/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215626891
Actually it seems like some files are missing?
For example, ThriftBinaryCLIService.java is not there, but it is in Hive.
---
If your project is set up for it, yo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12769#issuecomment-215626863
**[Test build #57303 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57303/consoleFull)**
for PR 12769 at commit
[`308a896`](https://g
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12259#discussion_r61534345
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/UDTRegistration.scala ---
@@ -0,0 +1,89 @@
+/*
+ * Licensed to the Apache Software Fo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215626610
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12770#issuecomment-215626624
**[Test build #57309 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57309/consoleFull)**
for PR 12770 at commit
[`2a84533`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215626609
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12773#issuecomment-215626453
**[Test build #57300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57300/consoleFull)**
for PR 12773 at commit
[`ea2ba20`](https://g
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/12774#discussion_r61534248
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/fileSourceInterfaces.scala
---
@@ -376,14 +376,10 @@ class HDFSFileCatalog(
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12764#issuecomment-215625985
Whenever you change the dependencies you'd need to change the file I think.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12738#issuecomment-215625153
**[Test build #57308 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57308/consoleFull)**
for PR 12738 at commit
[`161aa91`](https://gi
1 - 100 of 850 matches
Mail list logo