Github user wkhapy commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-85978470
@marmbrus ï¼when I use Statement.cancel() from JDBC,web ui shows that
query keep going ã
---
If your project is set up for it, you can reply to this email and have yo
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3718
---
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 enab
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-69479191
Thanks, 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 th
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-69476551
ping @marmbrus
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-69181757
This 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
en
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-68408513
@marmbrus, can this go since i have filed a PR to add pop-up for full job
desc?
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-68235672
Filed a PR to add pop-up for full job description #3819
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67923806
I see. Yes, a random group ID string is not intuitive nor useful. And just
realize another reason that we shouldn't set group ID in
`SparkExecuteStatementOperation.run`
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67886480
It's not deprecated for any good reason and I think we should undeprecate
it.
I'm not opposed to setting the job group, but I am opposed to setting it to
a rand
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67806102
@marmbrus Maybe I missed something here, but why do we prefer
`setJobDescription` over `setJobGroup`, especially when the former is
deprecated?
@scwf For job c
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67797406
Ok, i will add a pop-up containing the full
---
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 markhamstra commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67750241
That wrapping in the UI is barely tolerable for what are still fairly
modest-length queries. We're really going to need some kind of elided query in
the main Jobs ta
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67746894
1 My locally test it seems ctrl-c stops(not cancel) a running sql.
2 The sql statement wraps in ui auto.
` from JD
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67733650
i think this is ok. Even if we set the job group id, we can not use the
cancel api to cancel job in spark-sql or beeline.
---
If your project is set up for it, you can repl
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67727829
LGTM - but are we sure we don't want to set the job group id? We can use
that to cancel running commands.
---
If your project is set up for it, you can reply to this email
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67695031
Lets wait for some feedback from @rxin or @pwendell on the APIs in
question here.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67604099
@marmbrus, is this ok to go?
---
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/3718#issuecomment-67603749
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67603746
[Test build #24625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24625/consoleFull)
for PR 3718 at commit
[`e0d6b5d`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67598149
[Test build #24625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24625/consoleFull)
for PR 3718 at commit
[`e0d6b5d`](https://githu
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67592957
Updated
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67468412
[Test build #24580 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24580/consoleFull)
for PR 3718 at commit
[`4d2038a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67468425
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67456523
[Test build #24580 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24580/consoleFull)
for PR 3718 at commit
[`4d2038a`](https://githu
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67456216
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67442452
[Test build #24570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24570/consoleFull)
for PR 3718 at commit
[`4d2038a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67442456
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67437137
[Test build #24570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24570/consoleFull)
for PR 3718 at commit
[`4d2038a`](https://githu
Github user scwf commented on a diff in the pull request:
https://github.com/apache/spark/pull/3718#discussion_r22017818
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/AbstractSparkSQLDriver.scala
---
@@ -53,6 +54,8 @@ private[hive] abstract
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/3718#discussion_r21993324
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/AbstractSparkSQLDriver.scala
---
@@ -53,6 +54,8 @@ private[hive] abstr
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67304700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67304693
[Test build #24540 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24540/consoleFull)
for PR 3718 at commit
[`df79837`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67296740
[Test build #24540 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24540/consoleFull)
for PR 3718 at commit
[`df79837`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67294522
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67294519
[Test build #24539 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24539/consoleFull)
for PR 3718 at commit
[`92ce834`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67294126
[Test build #24539 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24539/consoleFull)
for PR 3718 at commit
[`92ce834`](https://githu
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67293587
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282956
[Test build #24532 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24532/consoleFull)
for PR 3718 at commit
[`92ce834`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282950
[Test build #24532 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24532/consoleFull)
for PR 3718 at commit
[`92ce834`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282957
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282772
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282305
[Test build #24531 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24531/consoleFull)
for PR 3718 at commit
[`92ce834`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282295
[Test build #24531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24531/consoleFull)
for PR 3718 at commit
[`92ce834`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67282309
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
GitHub user scwf opened a pull request:
https://github.com/apache/spark/pull/3718
[SPARK-4871][SQL] Show sql statement in spark ui when run sql with spark-sql
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scwf/spark sparksqlui
47 matches
Mail list logo
