Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11484#issuecomment-191642597
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/11484#issuecomment-191642595
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/11203#issuecomment-191642847
**[Test build #52381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52381/consoleFull)**
for PR 11203 at commit
[`ecd1df1`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11203#issuecomment-191643227
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11283#issuecomment-191643198
**[Test build #52373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52373/consoleFull)**
for PR 11283 at commit
[`749be1b`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11203#issuecomment-191643226
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/11372#issuecomment-191643447
**[Test build #52383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52383/consoleFull)**
for PR 11372 at commit
[`2a469e8`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11283#issuecomment-191643566
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/11283#issuecomment-191643564
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/11487#issuecomment-191643706
**[Test build #52375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52375/consoleFull)**
for PR 11487 at commit
[`389d644`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11487#issuecomment-191643939
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/11487#issuecomment-191643944
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 davies commented on the pull request:
https://github.com/apache/spark/pull/11341#issuecomment-191644061
LGTM, merging this 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 d
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9226#discussion_r54846913
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -218,25 +233,19 @@ private[hive] class HiveMetastoreCatalog(va
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191644225
**[Test build #52384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52384/consoleFull)**
for PR 11464 at commit
[`d2ace37`](https://gi
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191644382
test 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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/11341
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9226#discussion_r54847179
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -218,25 +233,19 @@ private[hive] class HiveMetastoreCatalog(va
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191646312
**[Test build #52385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52385/consoleFull)**
for PR 11464 at commit
[`e4d4cfb`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191646672
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191648049
**[Test build #52386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52386/consoleFull)**
for PR 11485 at commit
[`9db97ec`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191648669
**[Test build #52378 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52378/consoleFull)**
for PR 11464 at commit
[`2817c9a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191648913
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/11464#issuecomment-191648914
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/11487#issuecomment-191649365
I assume it still works in 2.11, and is at worst redundant but doesn't hurt
much. This is definitely the right lock? Maybe a comment pointing to why that
is correct woul
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191649872
**[Test build #52387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52387/consoleFull)**
for PR 11443 at commit
[`69695fb`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-191654553
**[Test build #52388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52388/consoleFull)**
for PR 11301 at commit
[`c763975`](https://gi
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191661797
@rxin We don't. As what have been done in this PR, we only replaces
`DataFrame` with `Dataset` on Java side.
---
If your project is set up for it, you can reply to t
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11488#issuecomment-191664081
OK, I'll merge when tests pass here, 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 p
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11223#issuecomment-191669921
This needs a few things -- a rebase of course.
Next I'm concerned with dependency harmonization. For example I see
differing versions of Jackson dependencies be
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11481#issuecomment-191670172
**[Test build #52369 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52369/consoleFull)**
for PR 11481 at commit
[`dd43417`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11481#issuecomment-191670305
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/11481#issuecomment-191670308
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/11481#issuecomment-191671137
Jenkins, test 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 t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11481#issuecomment-191672931
**[Test build #52389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52389/consoleFull)**
for PR 11481 at commit
[`dd43417`](https://gi
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11477#discussion_r54853932
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ---
@@ -320,9 +320,11 @@ private[hive] trait HiveInspectors {
case
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11372#issuecomment-191678584
**[Test build #52383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52383/consoleFull)**
for PR 11372 at commit
[`2a469e8`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11372#issuecomment-191678874
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/11372#issuecomment-191678876
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 yanboliang commented on the pull request:
https://github.com/apache/spark/pull/11266#issuecomment-191680086
@yinxusen I notice that your change made ```Pipeline``` and
```PipelineModel``` as the wrapper of peer Java object. But I vote to keep the
exist behavior, ```Pipelin
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191680443
**[Test build #52386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52386/consoleFull)**
for PR 11485 at commit
[`9db97ec`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191680718
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191680716
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/11478#issuecomment-191681417
**[Test build #2605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2605/consoleFull)**
for PR 11478 at commit
[`4c6e791`](https://
GitHub user devaraj-kavali opened a pull request:
https://github.com/apache/spark/pull/11490
[SPARK-13117] [Web UI] WebUI should use the local ip not 0.0.0.0
## What changes were proposed in this pull request?
In WebUI, now Jetty Server starts with SPARK_PUBLIC_DNS config va
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11490#issuecomment-191681962
Can one of the admins verify this patch?
---
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 p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11488#issuecomment-191683625
**[Test build #2606 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2606/consoleFull)**
for PR 11488 at commit
[`dd66979`](https://
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/11203#issuecomment-191683786
LGTM for me, cc @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 not hav
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11488#issuecomment-191685036
Merged 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 featu
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11292#issuecomment-191686172
Merged 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 featu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11488
---
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/11292
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191690696
**[Test build #52385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52385/consoleFull)**
for PR 11464 at commit
[`e4d4cfb`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191691005
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/11464#issuecomment-191691003
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 eliasah opened a pull request:
https://github.com/apache/spark/pull/11491
[SPARK-13646] [MLlib] QuantileDiscretizer counts dataset twice in getâ¦
## What changes were proposed in this pull request?
It avoids counting the dataframe twice.
You can merge thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191691667
Can one of the admins verify this patch?
---
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 p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191691807
**[Test build #52390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52390/consoleFull)**
for PR 11443 at commit
[`2bc521a`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191691777
**[Test build #52384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52384/consoleFull)**
for PR 11464 at commit
[`d2ace37`](https://g
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191691673
Jenkins test 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 th
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11464#issuecomment-191692104
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/11464#issuecomment-191692108
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 dragos commented on a diff in the pull request:
https://github.com/apache/spark/pull/11369#discussion_r54859540
--- Diff:
core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcherArguments.scala
---
@@ -44,7 +44,7 @@ private[mesos] class
MesosClusterDis
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/11492
[SPARK-13647][SQL] also check if numeric value is within allowed range in
_verify_type
## What changes were proposed in this pull request?
This PR makes the `_verify_type` in `types.py`
Github user dragos commented on the pull request:
https://github.com/apache/spark/pull/11369#issuecomment-191692543
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11492#issuecomment-191692842
**[Test build #52391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52391/consoleFull)**
for PR 11492 at commit
[`23222b6`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191692846
**[Test build #52392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52392/consoleFull)**
for PR 11491 at commit
[`0f319ed`](https://gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11438#issuecomment-191692936
Merged 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 featu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191693151
**[Test build #52392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52392/consoleFull)**
for PR 11491 at commit
[`0f319ed`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191693158
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191693154
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/11292#issuecomment-191694216
Nuts. I also just got bitten by 2.10 compatibility in two changes:
```
[error]
/home/jenkins/workspace/spark-master-compile-maven-scala-2.10/core/src/main/sc
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/11486#issuecomment-191694969
Labels of ML ```NaiveBayesModel``` are sorted(FYI #7284), so we do not need
to store it as member variable. Then it can pass the binary compatibility check.
---
If
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-191696160
**[Test build #52388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52388/consoleFull)**
for PR 11301 at commit
[`c763975`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-191696676
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/11301#issuecomment-191696679
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 eliasah commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191696680
I'm not sure why the *scala style tests" fails.
---
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 user srowen opened a pull request:
https://github.com/apache/spark/pull/11493
[SPARK-13423] [HOTFIX] Static analysis fixes for 2.x / fixed for Scala 2.10
## What changes were proposed in this pull request?
Fixes compile problem due to inadvertent use of `Option.contai
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/11487#issuecomment-191696955
This PR can resolve the issue mentioned in
[SPARK-10719](https://issues.apache.org/jira/browse/SPARK-10719) right?
---
If your project is set up for it, you can reply
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191697011
It's in the test output -- stray space before the comma :)
---
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 sarutak commented on the pull request:
https://github.com/apache/spark/pull/11487#issuecomment-191697316
@zsxwing who reported SPARK-10719.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11493#issuecomment-191698101
**[Test build #52393 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52393/consoleFull)**
for PR 11493 at commit
[`217dde3`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11492#issuecomment-191698467
**[Test build #52391 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52391/consoleFull)**
for PR 11492 at commit
[`23222b6`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11492#issuecomment-191698743
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191699392
**[Test build #52387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52387/consoleFull)**
for PR 11443 at commit
[`69695fb`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191699744
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191699741
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 eliasah commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191699932
Ok, sorry this is my first PR. Should I correct, push and pr again ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191703162
Yes just push a tiny commit to the same branch
---
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 eliasah commented on the pull request:
https://github.com/apache/spark/pull/11491#issuecomment-191703696
Done !
---
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
enable
Github user kiszk commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-191720188
Addressed all of CRs
---
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 fe
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11492#issuecomment-191726473
cc @yhuai @davies
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11492#discussion_r54870946
--- Diff: python/pyspark/sql/types.py ---
@@ -1137,7 +1148,28 @@ def _verify_type(obj, dataType):
if type(obj) not in _acceptable_types[_type]:
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191726573
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-191726855
**[Test build #52394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52394/consoleFull)**
for PR 11485 at commit
[`9db97ec`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191729769
**[Test build #52390 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52390/consoleFull)**
for PR 11443 at commit
[`2bc521a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191729946
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-191729943
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 steveloughran commented on the pull request:
https://github.com/apache/spark/pull/11449#issuecomment-191741549
The risk is deserialization; Groovy CVE-2015-3253 shows how groovy < 2.4.4
makes it straightforward to use a class in Groovy to run arbitrary shell
commands on t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11493#issuecomment-191743759
**[Test build #52393 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52393/consoleFull)**
for PR 11493 at commit
[`217dde3`](https://g
1 - 100 of 611 matches
Mail list logo