Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
Oh, you mean the the following. I'll fix this PR again. Sorry, @gatorsmile .
```scala
private val baseResourcePath = {
// If regenerateGoldenFiles is true, we must be
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15789
**[Test build #68233 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68233/consoleFull)**
for PR 15789 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
It's fixed like `SQLQueryTestSuite`.
---
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 kiszk commented on the issue:
https://github.com/apache/spark/pull/15745
Jenkins, 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 feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68233/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68235 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68235/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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/15780
**[Test build #68229 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68229/consoleFull)**
for PR 15780 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15780
**[Test build #68230 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68230/consoleFull)**
for PR 15780 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15780
**[Test build #68229 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68229/consoleFull)**
for PR 15780 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15789
**[Test build #68232 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68232/consoleFull)**
for PR 15789 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15745
**[Test build #68234 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68234/consoleFull)**
for PR 15745 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68232/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15780
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68229/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15780
**[Test build #68230 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68230/consoleFull)**
for PR 15780 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15780
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15789
**[Test build #68233 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68233/consoleFull)**
for PR 15789 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
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/15789
**[Test build #68237 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68237/consoleFull)**
for PR 15789 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11105
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68224/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15767
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/15767
**[Test build #68225 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68225/consoleFull)**
for PR 15767 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11105
**[Test build #68224 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68224/consoleFull)**
for PR 11105 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15767
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68225/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15745
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15745
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68226/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15745
**[Test build #68226 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68226/consoleFull)**
for PR 15745 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15788
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15788
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68228/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15788
**[Test build #68228 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68228/consoleFull)**
for PR 15788 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/15546#discussion_r86680401
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala
---
@@ -45,7 +45,7 @@ class LogicalPlanToSQLSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15618
**[Test build #68231 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68231/consoleFull)**
for PR 15618 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
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 feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15745
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15618
Hm.. @srowen @tdas and @mridulm . I just got rid of the changes in
`ReceiverTracker.scala` here for now. It seems we can't make this `null` in
`stop()` and create an instance in `start()`
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
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 feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15546
**[Test build #68239 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68239/consoleFull)**
for PR 15546 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15789
**[Test build #68232 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68232/consoleFull)**
for PR 15789 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11105
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68227/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
Hi, @gatorsmile .
I create a PR for `LogicalPlanToSQLSuite` part from #15546 .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
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 feature
enabled and
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
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 feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68235 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68235/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15767
**[Test build #68236 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68236/consoleFull)**
for PR 15767 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15742
**[Test build #68238 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68238/consoleFull)**
for PR 15742 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15745
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68234/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15745
**[Test build #68234 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68234/consoleFull)**
for PR 15745 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68235/
Test FAILed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15546
Hi, @gatorsmile . The PR is updated according to the advice.
- Remove the `goldenSQLPath` from this PR.
- Update the PR description about the target fixed cases and the detailed
plans,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
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/15618
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68231/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15788
**[Test build #68228 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68228/consoleFull)**
for PR 15788 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15546
Thank you for review, @gatorsmile . I'll add the failed plans, 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15767
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 kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r86680209
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -50,7 +50,7 @@ object ExpressionEncoder {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15618
**[Test build #68231 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68231/consoleFull)**
for PR 15618 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15767#discussion_r86680644
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -232,7 +232,11 @@ private[csv] object
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11105
**[Test build #68227 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68227/consoleFull)**
for PR 11105 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11105
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 dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/15789
[SPARK-18292][SQL] LogicalPlanToSQLSuite should not use resource dependent
path for golden file generation
## What changes were proposed in this pull request?
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15789
There occurs four `randomized aggregation test`s failures due to
`OutOfMemoryError`. But, it seems to be irrelevant. I'll wait the next test
running currently.
```
[info] - randomized
Github user vijoshi commented on the issue:
https://github.com/apache/spark/pull/15742
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 feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15767
**[Test build #68236 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68236/consoleFull)**
for PR 15767 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15767
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68236/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15789
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68237/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15789
**[Test build #68237 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68237/consoleFull)**
for PR 15789 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15789#discussion_r86688317
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala
---
@@ -46,7 +46,15 @@ class LogicalPlanToSQLSuite extends
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15484#discussion_r86688650
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -216,10 +216,16 @@ class Analyzer(
*
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15764#discussion_r86689510
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -923,6 +923,24 @@ class SessionCatalog(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15652#discussion_r86686887
--- Diff: docs/configuration.md ---
@@ -1664,6 +1664,15 @@ Apart from these, the following properties are also
available, and may be useful
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15618#discussion_r86687384
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/MapWithStateSuite.scala ---
@@ -38,19 +38,16 @@ class MapWithStateSuite extends SparkFunSuite
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15484#discussion_r86688702
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -229,10 +235,14 @@ class Analyzer(
*
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15759
---
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 tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15652#discussion_r86689289
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -308,14 +308,22 @@ private[spark] object JettyUtils extends Logging {
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15787
Jenkins add to whitelist
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15742
**[Test build #68238 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68238/consoleFull)**
for PR 15742 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15742
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/15742
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68238/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15546
**[Test build #68239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68239/consoleFull)**
for PR 15546 at commit
Github user fidato13 commented on the issue:
https://github.com/apache/spark/pull/15327
ping @rxin @srowen @zsxwing The build and tests have passed. Request you to
have a look and merge as you get time. Thanks!
---
If your project is set up for it, you can reply to this email and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68240/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68240/
Test FAILed.
---
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15652#discussion_r86689337
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -308,14 +308,22 @@ private[spark] object JettyUtils extends Logging {
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15432
---
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/12257
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/15759
LGTM, merged into master and branch-2.1. 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68240 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68240/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15688
**[Test build #68241 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68241/consoleFull)**
for PR 15688 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68242 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68242/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15546
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68239/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15546
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 srowen commented on the issue:
https://github.com/apache/spark/pull/15432
Merged to master/2.1
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15788
**[Test build #68243 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68243/consoleFull)**
for PR 15788 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15652#discussion_r86686861
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -308,14 +308,22 @@ private[spark] object JettyUtils extends Logging {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15789
LGTM except a 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 sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
Not my day...
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 feature
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15546
@dongjoon-hyun I have a general comment about the SQL generation fixes. Not
sure how you fix the issue. Normally, when we did it, we need to see the
analyzed SQL plan. In this case, it becomes a
1 - 100 of 330 matches
Mail list logo
