Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104129596
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104146550
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104131186
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17149
**[Test build #73842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73842/testReport)**
for PR 17149 at commit
[`e792cb6`](https://github.com/apache/spark/commit/e
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104147006
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104152377
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104142173
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104125659
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104128753
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r104147505
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,274 @@
+/*
+ * Licensed
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/17080
thanks. One thing I realised last night is that logging the session token,
even at debug level, would have been a security risk. So it's very good that
the log statement got cut, even at the c
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17143
Thank you for merging, @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
enab
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16696
**[Test build #73845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73845/testReport)**
for PR 16696 at commit
[`516b114`](https://github.com/apache/spark/commit/51
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17150
The context bound changes won't work on the two public methods:
```
[error] * deprecated method
accumulableCollection(java.lang.Object,scala.Function1,scala.reflect.ClassTag)org.apache.s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17114
**[Test build #73844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73844/testReport)**
for PR 17114 at commit
[`a981ed5`](https://github.com/apache/spark/commit/a9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17150
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73843/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17150
**[Test build #73843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73843/testReport)**
for PR 17150 at commit
[`933d180`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17150
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/17150
**[Test build #73843 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73843/testReport)**
for PR 17150 at commit
[`933d180`](https://github.com/apache/spark/commit/93
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17150#discussion_r104146216
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1391,8 +1391,9 @@ class SparkContext(config: SparkConf) extends Logging
{
*
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17150#discussion_r104146071
--- Diff: R/pkg/DESCRIPTION ---
@@ -54,5 +54,5 @@ Collate:
'types.R'
'utils.R'
'window.R'
-RoxygenNote: 5.0.1
+RoxygenNote:
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17150#discussion_r104146299
--- Diff: dev/change-version-to-2.11.sh ---
@@ -1,23 +0,0 @@
-#!/usr/bin/env bash
--- End diff --
These scripts were deprecated wrappers
--
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17150#discussion_r104146264
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -66,7 +66,7 @@ private[spark] object JettyUtils extends Logging {
implicit de
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/17150
[SPARK-19810][WIP][BUILD][CORE] Remove support for Scala 2.10
## What changes were proposed in this pull request?
- Remove Scala 2.10 build profiles and support
- Remove deprecated calls
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17149
**[Test build #73842 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73842/testReport)**
for PR 17149 at commit
[`e792cb6`](https://github.com/apache/spark/commit/e7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17149
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73841/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17149
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/17133
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73839/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17133
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/17149
**[Test build #73841 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73841/testReport)**
for PR 17149 at commit
[`890327a`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17133
**[Test build #73839 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73839/testReport)**
for PR 17133 at commit
[`38f9df2`](https://github.com/apache/spark/commit/3
Github user windpiger closed the pull request at:
https://github.com/apache/spark/pull/16619
---
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 windpiger commented on the issue:
https://github.com/apache/spark/pull/16619
yes I will close it , thanks! @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
e
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/17031
@srown There are parts for refactoring only purposes.
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/16619
Should this be closed in favor of
https://github.com/apache/spark/pull/17149 ?
---
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17133#discussion_r104136933
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskInfo.scala ---
@@ -75,6 +75,8 @@ class TaskInfo(
}
private[spark] def markFin
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/17031
@mgummelt LGTM. Thanks fo rthe clarifications. @srowen can we get a merge?
---
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 proj
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17148
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73837/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17148
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/17148
**[Test build #73837 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73837/testReport)**
for PR 17148 at commit
[`d4787b8`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73838/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
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/17001
**[Test build #73838 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73838/testReport)**
for PR 17001 at commit
[`7739ccd`](https://github.com/apache/spark/commit/7
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17143
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/17143
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 this feature
enabled and wishes so, or i
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17137
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/17137
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17080
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/17080
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 this feature
enabled and wishes so, or i
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17135
Yes this is substantially not something we can merge, so let's close this.
---
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 proj
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17149
**[Test build #73841 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73841/testReport)**
for PR 17149 at commit
[`890327a`](https://github.com/apache/spark/commit/89
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
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/17144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73833/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73833 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73833/testReport)**
for PR 17144 at commit
[`9ec5caf`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73836/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
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/17136
**[Test build #73836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73836/testReport)**
for PR 17136 at commit
[`d88a966`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73835/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16981
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/16981
**[Test build #73835 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73835/testReport)**
for PR 16981 at commit
[`4efae36`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73834/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17001
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/17001
**[Test build #73834 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73834/testReport)**
for PR 17001 at commit
[`ae9938a`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17149
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73840/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17149
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/17149
**[Test build #73840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73840/testReport)**
for PR 17149 at commit
[`69a1646`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17145
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73832/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17149
**[Test build #73840 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73840/testReport)**
for PR 17149 at commit
[`69a1646`](https://github.com/apache/spark/commit/69
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17145
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 windpiger opened a pull request:
https://github.com/apache/spark/pull/17149
[SPARK-19257][SQL][WIP]location for table/partition/database should be
java.net.URI
## What changes were proposed in this pull request?
Currently we treat the location of table/partition
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17145
**[Test build #73832 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73832/testReport)**
for PR 17145 at commit
[`f5a35f6`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17145
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/17145
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73830/
Test PASSed.
---
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17145
cc @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
enabled and wishes so, or if t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17145
**[Test build #73830 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73830/testReport)**
for PR 17145 at commit
[`f5a35f6`](https://github.com/apache/spark/commit/f
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17125#discussion_r104121538
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -604,7 +604,13 @@ class Analyzer(
def
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
cc @kayousterhout
---
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 wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104116975
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/StatsConfSuite.scala
---
@@ -1,64 +0,0 @@
-/*
- * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17133
**[Test build #73839 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73839/testReport)**
for PR 17133 at commit
[`38f9df2`](https://github.com/apache/spark/commit/38
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104116520
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/StatsEstimationSuite.scala
---
@@ -0,0 +1,121 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16910
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73831/
Test PASSed.
---
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104116400
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/StatisticsCollectionSuite.scala ---
@@ -116,22 +116,22 @@ class StatisticsCollectionSuite extends
Stat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16910
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/16910
**[Test build #73831 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73831/testReport)**
for PR 16910 at commit
[`15c0a77`](https://github.com/apache/spark/commit/1
Github user AnthonyTruchet commented on a diff in the pull request:
https://github.com/apache/spark/pull/17076#discussion_r104115651
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -440,19 +440,9 @@ private class LinearSVCAggregator(
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17148
cc @cloud-fan @ron8hu please review
---
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 a
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17114#discussion_r104115235
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveInlineTables.scala
---
@@ -95,10 +95,14 @@ object ResolveInlineTable
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17001
**[Test build #73838 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73838/testReport)**
for PR 17001 at commit
[`7739ccd`](https://github.com/apache/spark/commit/77
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17148
**[Test build #73837 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73837/testReport)**
for PR 17148 at commit
[`d4787b8`](https://github.com/apache/spark/commit/d4
GitHub user wzhfy opened a pull request:
https://github.com/apache/spark/pull/17148
[SPARK-17075][SQL][followup] fix filter estimation issues
## What changes were proposed in this pull request?
1. support boolean type in binary expression estimation.
2. don't support com
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17114#discussion_r104114853
--- Diff: sql/core/src/test/resources/sql-tests/inputs/inline-table.sql ---
@@ -29,6 +29,9 @@ select * from values ("one", array(0, 1)), ("two",
array(2,
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17114#discussion_r104114722
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveInlineTables.scala
---
@@ -95,10 +95,14 @@ object ResolveInlineTable
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16910#discussion_r104114101
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -1588,6 +1590,153 @@ class HiveDDLSuite
}
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17142
I definitely will. Thank you so much @cloud-fan and @maropu.
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16910#discussion_r104111452
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -1588,6 +1590,153 @@ class HiveDDLSuite
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r104111279
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -74,7 +88,17 @@ abstract class ExternalCatalog {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17142
---
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/17142
@HyukjinKwon you can address @maropu 's comments in your next CSV PR.
---
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 projec
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17142
thanks, 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 wi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17136
**[Test build #73836 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73836/testReport)**
for PR 17136 at commit
[`d88a966`](https://github.com/apache/spark/commit/d8
501 - 600 of 644 matches
Mail list logo