Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14690
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14690
**[Test build #66861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66861/consoleFull)**
for PR 14690 at commit
[`59fecdf`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14690
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66861/
Test FAILed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15427#discussion_r83140093
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1878,17 +1878,25 @@ class Dataset[T] private[sql](
def dropDuplicates(colNa
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15455
Merging in. Thanks.
Can you also close the pr? GitHub wont' close it automatically because it
is not merged into master branch.
---
If your project is set up for it, you can reply to this em
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66866/consoleFull)**
for PR 9766 at commit
[`18fa6e3`](https://github.com/apache/spark/commit/18f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66866/consoleFull)**
for PR 9766 at commit
[`18fa6e3`](https://github.com/apache/spark/commit/18
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66866/
Test FAILed.
---
I
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
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
en
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15435
So I've been reading through some of the history with logistic regression
summaries. There was a lot of discussion on how to design the abstractions for
this, [here](https://github.com/apache/spark/p
Github user dafrista closed the pull request at:
https://github.com/apache/spark/pull/15455
---
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 e
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15457#discussion_r83140489
--- Diff: sql/core/src/main/java/org/apache/spark/sql/api/java/UDF1.java ---
@@ -19,14 +19,12 @@
import java.io.Serializable;
-// **
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15457#discussion_r83140501
--- Diff: sql/core/src/main/java/org/apache/spark/sql/api/java/UDF1.java ---
@@ -19,14 +19,12 @@
import java.io.Serializable;
-// **
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/15457
[SPARK-17830][SQL] Annotate remaining SQL APIs with InterfaceStability
## What changes were proposed in this pull request?
This patch annotates all the remaining APIs in SQL (excluding streaming)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15457
**[Test build #66867 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66867/consoleFull)**
for PR 15457 at commit
[`5f51cbb`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66868/consoleFull)**
for PR 9766 at commit
[`00f65cd`](https://github.com/apache/spark/commit/00f
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15435
So, based on my interpretation of this and how this can actually work, we
need to have:
scala
sealed trait LogisticRegressionSummary
sealed trait LogisticRegressionTrainingSummary
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66868/consoleFull)**
for PR 9766 at commit
[`00f65cd`](https://github.com/apache/spark/commit/00
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
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
en
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66868/
Test FAILed.
---
I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15457
**[Test build #66869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66869/consoleFull)**
for PR 15457 at commit
[`9f7db6f`](https://github.com/apache/spark/commit/9
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/14690#discussion_r83141382
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -616,6 +617,44 @@ private[spark] class HiveExternalCatalog(conf:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15432
Unfortunately, not all the things have a standard to follow. That is why I
suggested you to do a research about it. Like Oracle, it does not have such a
function in their SQL-function list:
http
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66870 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66870/consoleFull)**
for PR 9766 at commit
[`8171b85`](https://github.com/apache/spark/commit/817
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15432
Strictly, the JIRA describes handling `null` and we might not have to
generalize the cases further.
> it will failed when do select rand(null)
Also, I would like to add the edge
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15408
**[Test build #66860 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66860/consoleFull)**
for PR 15408 at commit
[`b74fb36`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15408
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66860/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15408
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
e
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15432
At first, we do not strictly follow Hive. You can easily find many in
Spark. I do not think this is an urgent JIRA, right? Like what @srowen replied
in the JIRA, he does not think this is a bug.
Github user priyankagargnitk commented on the issue:
https://github.com/apache/spark/pull/15449
Thanks rxin
---
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,
Github user tnachen commented on the issue:
https://github.com/apache/spark/pull/12933
I just tried running it locally and I'm getting the same error. It seems
like with your change that test is simply declining the offer.
---
If your project is set up for it, you can reply to this e
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15432
Not urgent but in my experience such PR have been being hold. So, I am
trying to fix the problem specified in the JIRA only rather than fixing others
together.
@srowen said "I'm not ev
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15432
What is the behavior of `PostgreSQL`? Treating `NULL` as zero?
---
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 wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15365#discussion_r83143464
--- Diff: R/pkg/R/mllib.R ---
@@ -647,6 +654,195 @@ setMethod("predict", signature(object =
"KMeansModel"),
predict_internal(object, ne
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15456
**[Test build #66862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66862/consoleFull)**
for PR 15456 at commit
[`98e7015`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15456
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15456
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66862/
Test PASSed.
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15437
LGTM. Thanks! Merging to master and 2.0.
---
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
en
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15432
Ah, PostgreSQL seems unsetting the seed.
```
postgres=# SELECT setseed(0), random(), random();
setseed | random | random
-+---+---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15437
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15432
Now, at least we have four options, when users setting `NULL` as a seed
number for `rand`:
1. Hive/MySQL - `NULL` is equivalent to `0`
2. DB2 - when the seed is `NULL`, `rand` returns `NUL
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15458
cc @yhuai
---
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 th
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15458
[SPARK-17899][SQL] add a debug mode to keep raw table properties in
HiveExternalCatalog
## What changes were proposed in this pull request?
Currently `HiveExternalCatalog` will filter ou
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15458
**[Test build #66871 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66871/consoleFull)**
for PR 15458 at commit
[`e821f1a`](https://github.com/apache/spark/commit/e
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15458
LGTM pending Jenkins.
---
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,
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil looks like there is a legitimate failing test.
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15272
hm looks like another legitimate failing test too
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15427
LGTM, merging 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 feature
enabled and wish
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15427
---
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 viirya commented on the issue:
https://github.com/apache/spark/pull/15427
Thanks for review! @rxin @cloud-fan
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r83146607
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15230
LGTM except one minor comment
---
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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15432
Oh, strictly, it does not ignore in case of `PostgreSQL`. It unsets.
```sql
postgres=# SELECT setseed(0);
setseed
-
(1 row)
postgres=# SELECT rando
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/15456#discussion_r83146972
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -104,6 +104,8 @@ object SparkSubmit {
/___/ .__/\_,_/_/ /_/\_\ vers
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15432
Given different databases diverge (they don't even have the same function
names), I think it's fine to just have null be treated as 0 like Hive/MySQL.
---
If your project is set up for it, you can re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66865 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66865/consoleFull)**
for PR 15285 at commit
[`bd47bd4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66865/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15452
Merging in master/branch-2.0.
---
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 wis
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15414
**[Test build #66872 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66872/consoleFull)**
for PR 15414 at commit
[`7e2d501`](https://github.com/apache/spark/commit/7
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15452
---
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 issue:
https://github.com/apache/spark/pull/15416
**[Test build #3337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3337/consoleFull)**
for PR 15416 at commit
[`69f6e4f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15416
**[Test build #3337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3337/consoleFull)**
for PR 15416 at commit
[`69f6e4f`](https://github.com/apache/spark/commit
Github user loneknightpy commented on the issue:
https://github.com/apache/spark/pull/15285
@tdas Based on our offline discussion, I added file size cache for the
compressed log files.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15457
**[Test build #66869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66869/consoleFull)**
for PR 15457 at commit
[`9f7db6f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15457
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66869/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15457
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15457
**[Test build #66867 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66867/consoleFull)**
for PR 15457 at commit
[`5f51cbb`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15457
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66867/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15457
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
e
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15432
@HyukjinKwon Could you document the behavior in the description of `rand`
function? Checked whether we have any missing test case? Not sure whether you
are also can check whether `rand` in R and
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15230#discussion_r83149074
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -111,6 +111,10 @@ private[spark] class HiveExternalCatalog(co
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r83149648
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15148
**[Test build #66873 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66873/consoleFull)**
for PR 15148 at commit
[`a35e261`](https://github.com/apache/spark/commit/a
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15457
LGTM - pending jenkins
---
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 wishe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9766
**[Test build #66870 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66870/consoleFull)**
for PR 9766 at commit
[`8171b85`](https://github.com/apache/spark/commit/81
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15148
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66873/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15148
**[Test build #66873 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66873/consoleFull)**
for PR 15148 at commit
[`a35e261`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15148
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66870/
Test PASSed.
---
I
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9766
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
en
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15457
**[Test build #66874 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66874/consoleFull)**
for PR 15457 at commit
[`6f9b673`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15230
**[Test build #66876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66876/consoleFull)**
for PR 15230 at commit
[`330913e`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15456
**[Test build #66875 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66875/consoleFull)**
for PR 15456 at commit
[`3c2b729`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15307
**[Test build #66877 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66877/consoleFull)**
for PR 15307 at commit
[`49da4b1`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15307
**[Test build #66878 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66878/consoleFull)**
for PR 15307 at commit
[`839d402`](https://github.com/apache/spark/commit/8
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/15307
@marmbrus
Please review one more time! Other than addressing your comments, I also
found a few bugs which I added tests for.
---
If your project is set up for it, you can reply to this email and
601 - 687 of 687 matches
Mail list logo