Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17818
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/17818
**[Test build #76375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76375/testReport)**
for PR 17818 at commit
[`38f43d0`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76375/
Test PASSed.
---
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/17678
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 wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17702
yea I'm also wondering how useful it is
---
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/17818
**[Test build #76375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76375/testReport)**
for PR 17818 at commit
[`38f43d0`](https://github.com/apache/spark/commit/38
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17819#discussion_r114254089
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1883,6 +1883,56 @@ class Dataset[T] private[sql](
}
/**
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17826
---
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 the issue:
https://github.com/apache/spark/pull/17826
thanks, merging to master/2.2!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled an
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17770
I think we should have a single way to stop analyzing already analyzed
plans. We should either apply `resolveOperators` more widely, or switch to use
this analysis barrier.
---
If your project i
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17829
LGTM. Wait for more reviews. 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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17811
---
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 the issue:
https://github.com/apache/spark/pull/17811
LGTM, merging to master/2.2
---
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 w
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17540
The last commit fails a lot of tests...
---
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/17829
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/17829
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76371/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17829
**[Test build #76371 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76371/testReport)**
for PR 17829 at commit
[`c22568f`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17827
**[Test build #76374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76374/testReport)**
for PR 17827 at commit
[`6d658d4`](https://github.com/apache/spark/commit/6d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17825
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/17825
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76373/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17825
**[Test build #76373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76373/testReport)**
for PR 17825 at commit
[`d892f30`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17828
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76372/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17828
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/17828
**[Test build #76372 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76372/testReport)**
for PR 17828 at commit
[`fa53864`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17827
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/17827
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76370/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17827
**[Test build #76370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76370/testReport)**
for PR 17827 at commit
[`008cec4`](https://github.com/apache/spark/commit/0
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17825
That makes sense I guess. It would be great to have more control over the
layout though. One can dream, right? :)
Thank you so much for all the reviews and information.
---
If your project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17825
**[Test build #76373 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76373/testReport)**
for PR 17825 at commit
[`d892f30`](https://github.com/apache/spark/commit/d8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17829
**[Test build #76371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76371/testReport)**
for PR 17829 at commit
[`c22568f`](https://github.com/apache/spark/commit/c2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17828
**[Test build #76372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76372/testReport)**
for PR 17828 at commit
[`fa53864`](https://github.com/apache/spark/commit/fa
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/17715
@jkbradley Thanks for your comments. I sent #17829 to address them, please
feel free to review. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17715#discussion_r114248399
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -178,11 +178,86 @@ private[classification] trait
Logi
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17818#discussion_r114248432
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1656,6 +1656,18 @@ test_that("greatest() and least() on a DataFrame", {
expect_equal(colle
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/17829
[SPARK-20047][FOLLOWUP][ML] Constrained Logistic Regression follow up
## What changes were proposed in this pull request?
Address some minor comments for #17715:
* Put bound-constrained o
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/17828
[SPARK-20490][SPARKR][DOC] add family tag for not function
## What changes were proposed in this pull request?
doc only
## How was this patch tested?
manual
You can m
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17818
> hmm, not clear why AppVeyor failed. you could trigger it again by closing
and re-opening this PR
without affecting Jenkins
Look I'll have to rebase it anyway but thank you so much for
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/12646
retest 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 wishes so, or i
Github user zero323 closed the pull request at:
https://github.com/apache/spark/pull/17818
---
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 en
GitHub user zero323 reopened a pull request:
https://github.com/apache/spark/pull/17818
[SPARK-20544] R wrapper for input_file_name
## What changes were proposed in this pull request?
Adds wrapper for `o.a.s.sql.functions.input_file_name`
## How was this patch teste
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17383
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 project does not have this feat
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17807
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/17807
merged 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 and w
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17818#discussion_r114246581
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1656,6 +1656,18 @@ test_that("greatest() and least() on a DataFrame", {
expect_equal(c
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17818#discussion_r114246336
--- Diff: R/pkg/R/functions.R ---
@@ -3890,3 +3890,23 @@ setMethod("not",
jc <- callJStatic("org.apache.spark.sql.functions", "not",
x@
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17818
hmm, not clear why AppVeyor failed. you could trigger it again by closing
and re-opening this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17825#discussion_r114245870
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3715,3 +3715,24 @@ setMethod("rollup",
sgd <- callJMethod(x@sdf, "rollup", jcol)
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17825#discussion_r114245853
--- Diff: R/pkg/R/column.R ---
@@ -132,16 +132,23 @@ createMethods()
#' alias
#'
-#' Set a new name for a column
+#' Set a new na
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17825#discussion_r114245818
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -2253,6 +2253,15 @@ test_that("mutate(), transform(), rename() and
names()", {
detach(
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17825#discussion_r114245756
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -2253,6 +2253,15 @@ test_that("mutate(), transform(), rename() and
names()", {
detach(
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17825#discussion_r114245780
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3715,3 +3715,24 @@ setMethod("rollup",
sgd <- callJMethod(x@sdf, "rollup", jcol)
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17816
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/17816
thanks
merged to master/2.2
---
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/17827
**[Test build #76370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76370/testReport)**
for PR 17827 at commit
[`008cec4`](https://github.com/apache/spark/commit/00
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17827
cc @gatorsmile and @ptkool, could you take a look and see if it makes sense
please?
---
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17827#discussion_r114244380
--- Diff: python/pyspark/sql/column.py ---
@@ -224,7 +224,39 @@ def __init__(self, jc):
https://spark.apache.org/docs/latest/sql-programming
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17827#discussion_r114244470
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala ---
@@ -284,23 +287,6 @@ class ColumnExpressionSuite extends QueryTes
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17827#discussion_r114244526
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala ---
@@ -284,23 +287,6 @@ class ColumnExpressionSuite extends QueryTes
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17827#discussion_r114244337
--- Diff: python/pyspark/sql/column.py ---
@@ -224,7 +224,39 @@ def __init__(self, jc):
https://spark.apache.org/docs/latest/sql-programming
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/17827
[SPARK-20552][SQL][PYTHON] Add isNotDistinctFrom/isDistinctFrom for column
APIs in Scala and Python
## What changes were proposed in this pull request?
This PR proposes to add both `is
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/17746
@dbtsai Thanks for the explanation and the context :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have thi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17540
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76367/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17540
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/17540
**[Test build #76367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76367/testReport)**
for PR 17540 at commit
[`6752ff2`](https://github.com/apache/spark/commit/6
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17770
It is possible as I think `resolveOperators` works as the same as this
analysis barrier + `transformUp`. However, `resolveOperators` is widely used
now so we may not have urgent need to remove it.
Github user leonfl commented on the issue:
https://github.com/apache/spark/pull/16486
@mrjrdnthms ,Yes, your understand is correct, in scala it like this:
```
val rows: RDD[Row] = df.rdd.map(
rowIn => {
// handle the rowIn and return a Row
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17540
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76368/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17540
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/17540
**[Test build #76368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76368/testReport)**
for PR 17540 at commit
[`90045cf`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17825
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76369/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17825
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/17825
**[Test build #76369 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76369/testReport)**
for PR 17825 at commit
[`7abe6ca`](https://github.com/apache/spark/commit/7
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114239173
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -1009,10 +1009,17 @@ private[spark] object RandomForest extends Loggin
Github user carsonwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17540#discussion_r114238664
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -73,21 +99,35 @@ object SQLExecution {
}
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114238091
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -1009,10 +1009,24 @@ private[spark] object RandomForest extends Loggin
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17824
Yes `SparkStatusTrack` will keep working the same way it does today.
---
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 do
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17723
It would be easier if instead you said what particular API that is being
proposed here you have issues with. Is it the storing of credentials in UGI? Or
what?
If that's what you're saying. I
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76366/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17826
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/17826
**[Test build #76366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76366/testReport)**
for PR 17826 at commit
[`6d59636`](https://github.com/apache/spark/commit/6
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17824
Thanks @vanzin ! `SparkStatusTracker` depends on `JobProgressListener`
which was already deprecated, will you remove this `JobProgressListener` and
rewrite `SparkStatusTrack`?
---
If your projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17825
**[Test build #76369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76369/testReport)**
for PR 17825 at commit
[`7abe6ca`](https://github.com/apache/spark/commit/7a
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17824
I'm planning to remove these listeners completely in 2.3. (StorageStatus,
which is the theme of this particular PR, is not currently in my crosshair, but
I my decide to clean that one up also.)
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17764
I filed [SPARK-20552](https://issues.apache.org/jira/browse/SPARK-20552)
for column APIs in both Python and Scala/Java. Do you mind if I work on this if
any of you, @gatorsmile and @ptkool, is c
Github user carsonwang commented on the issue:
https://github.com/apache/spark/pull/17540
Yes, that's reasonable. I was asking because I noticed `withNewExecutionId`
was added in `hiveResultString` method so it should have been fixed.
---
If your project is set up for it, you can rep
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17746
The motivation to have this one merged in Spark 2.2 is not only just for
#17715 but also because Breeze 0.13.x fixes many bugs in upstream. Since Spark
was tightened to 0.12, many users (including my
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17824
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/17824
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76364/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17824
**[Test build #76364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76364/testReport)**
for PR 17824 at commit
[`15d1977`](https://github.com/apache/spark/commit/1
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17824
@vanzin are we going to remove these listeners in future, or just keep them
as deprecated? Some projects like Zeppelin explicitly depends on these
listeners not only for code simplicity, but also
Github user carsonwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17540#discussion_r114235457
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -73,21 +99,35 @@ object SQLExecution {
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17540
**[Test build #76368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76368/testReport)**
for PR 17540 at commit
[`90045cf`](https://github.com/apache/spark/commit/90
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/17540#discussion_r114235126
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -73,21 +99,35 @@ object SQLExecution {
}
r
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17540
**[Test build #76367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76367/testReport)**
for PR 17540 at commit
[`6752ff2`](https://github.com/apache/spark/commit/67
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/17540
@carsonwang, the plan for when we notice queries that don't appear in the
SQL tab is to add a call to `checkSQLExecutionId`, which will cause tests to
fail when that operation isn't wrapped by `withN
Github user carsonwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17540#discussion_r114234728
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -73,21 +99,35 @@ object SQLExecution {
}
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17795
@vanzin , since branch 2.0 doesn't have this feature in UI
(https://issues.apache.org/jira/browse/SPARK-11272), so I don't think it is
required to fix in branch-2.0.
---
If your project is set u
Github user carsonwang commented on the issue:
https://github.com/apache/spark/pull/17540
Hi @rdblue , just wanted to confirm that this also fixed #17535 so we
should have the UI when executing queries in Spark SQL CLI?
---
If your project is set up for it, you can reply to this ema
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/17540
Rebased. I'll check if tests pass later tonight.
---
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 fea
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17819#discussion_r114234373
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1883,6 +1883,56 @@ class Dataset[T] private[sql](
}
/**
+
1 - 100 of 288 matches
Mail list logo