Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15171
**[Test build #65693 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65693/consoleFull)**
for PR 15171 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15170
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15170
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65691/
Test PASSed.
---
Github user clockfly commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r79752478
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -142,8 +145,12 @@ class SessionCatalog(
//
Github user clockfly commented on a diff in the pull request:
https://github.com/apache/spark/pull/14897#discussion_r79752499
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -36,6 +36,7 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15102
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65694/
Test PASSed.
---
Github user cenyuhai commented on a diff in the pull request:
https://github.com/apache/spark/pull/15041#discussion_r79753174
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -1384,14 +1385,15 @@ abstract class RDD[T: ClassTag](
* @param ord the implicit
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15059#discussion_r79755313
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
---
@@ -170,11 +170,9 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14644
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14644
**[Test build #65700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65700/consoleFull)**
for PR 14644 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14644
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65700/
Test FAILed.
---
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15131#discussion_r79755939
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1426,7 +1426,7 @@ class SparkContext(config: SparkConf) extends Logging
with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15171
**[Test build #65701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65701/consoleFull)**
for PR 15171 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65699/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65699/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14995
**[Test build #65695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65695/consoleFull)**
for PR 14995 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14995
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65696/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14995
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15173
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65697/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15173
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15174
**[Test build #65702 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65702/consoleFull)**
for PR 15174 at commit
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14959
I just fix the unti test, @vanzin @davies @holdenk Mind to help review it
? Thanks
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15170
So this pr actually doesn't merge cleanly into branch-2.0. Can you submit a
new pr for branch-2.0?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15170
---
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 user rxin commented on the issue:
https://github.com/apache/spark/pull/15170
I have merged this in master 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 project does not have this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15140
**[Test build #65652 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65652/consoleFull)**
for PR 15140 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65653 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65653/consoleFull)**
for PR 12819 at commit
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15160
[SPARK-17609][SQL] SessionCatalog.tableExists should not check temp view
## What changes were proposed in this pull request?
After #15054 , there is no place in Spark SQL that need
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15139
**[Test build #65656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65656/consoleFull)**
for PR 15139 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15139#discussion_r79611427
--- Diff: R/pkg/inst/tests/testthat/test_mllib.R ---
@@ -760,13 +760,7 @@ test_that("spark.kstest", {
expect_equal(stats$p.value,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15152
**[Test build #65655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65655/consoleFull)**
for PR 15152 at commit
Github user jlhitt commented on the issue:
https://github.com/apache/spark/pull/14762
BTW, any speed on SPARC is an improvement over not working at all... The
only regression to be worried about is x86.
---
If your project is set up for it, you can reply to this email and have your
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r79613964
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -38,15 +38,15 @@
*
* The
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15139#discussion_r79614619
--- Diff: R/pkg/R/mllib.R ---
@@ -1398,20 +1398,22 @@ setMethod("summary", signature(object = "KSTest"),
distParams <-
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15005
@frreiss are you in a position to finish this one?
---
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15150#discussion_r79605580
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala ---
@@ -580,10 +581,14 @@ class Word2VecModel private[spark] (
ind +=
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15140
@BryanCutler Updated, thanks your comments.
---
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 the issue:
https://github.com/apache/spark/pull/15160
cc @yhuai @gatorsmile
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65653/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65653 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65653/consoleFull)**
for PR 12819 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r79614191
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeMapData.java
---
@@ -67,9 +67,9 @@ public UnsafeMapData() {
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15131
@shivaram @felixcheung @HyukjinKwon Any thoughts? 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15160
**[Test build #65654 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65654/consoleFull)**
for PR 15160 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r79614500
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeArrayWriter.java
---
@@ -199,6 +200,7 @@ public void
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r79614390
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/UnsafeArraySuite.scala
---
@@ -141,12 +141,18 @@ class UnsafeArraySuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15139
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65656/
Test PASSed.
---
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/15149#discussion_r79628242
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/ProbabilisticClassifier.scala
---
@@ -200,26 +200,9 @@ abstract class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #65657 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65657/consoleFull)**
for PR 13680 at commit
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/15102
You should not be assuming 0 for a starting offset for partitions you've
just learned about. You should be asking the underlying driver consumer what
its position is. This is yet another
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
No problem. thanks very much @srowen .
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65658/consoleFull)**
for PR 14597 at commit
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/15149#discussion_r79630399
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -56,6 +56,21 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #65651 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65651/consoleFull)**
for PR 13680 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15143
This is to some degree covered by other tests, yes. It's probably just not
worth 100 lines of test code to test the contents of a Set from all angles.
I think the other two non-test changes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65651/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15139
**[Test build #65656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65656/consoleFull)**
for PR 15139 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15139
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 does not have this feature
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15054
It should be not hard. Will make a try today to backport to 2.0. 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
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15173
LGTM. Merging to 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 this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15175
**[Test build #65704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65704/consoleFull)**
for PR 15175 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15173
---
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15131
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65698/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15131
**[Test build #65705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65705/consoleFull)**
for PR 15131 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14971
**[Test build #65703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65703/consoleFull)**
for PR 14971 at commit
GitHub user Sherry302 opened a pull request:
https://github.com/apache/spark/pull/15175
[MINOR][BUILD] Fix CheckStyle Error
## What changes were proposed in this pull request?
This PR is to fix the code style errors.
## How was this patch tested?
Manual.
Github user Sherry302 commented on the issue:
https://github.com/apache/spark/pull/15170
@lresende @rxin Thank for the review. I have created a
[PR](https://github.com/apache/spark/pull/15175) to branch-2.0.
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15131
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15131
**[Test build #65698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65698/consoleFull)**
for PR 15131 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15131
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 this feature
enabled and
501 - 572 of 572 matches
Mail list logo