Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19717
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84453/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19717
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19717
**[Test build #84453 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84453/testReport)**
for PR 19717 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19871
**[Test build #84461 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84461/testReport)**
for PR 19871 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r154866923
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19871
**[Test build #84460 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84460/testReport)**
for PR 19871 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154865943
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19882
It has more lines, doesn't it? In any way, we need helper functions.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r154865049
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154864703
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends Logging
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/19889
No other algs output multi-column for now
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19882
> OrcTest will provide common helper functions and def format: String.
Instead of having `def format: String`, can we just add `beforeAll` and
`afterAll` in the test suites to set the
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154863367
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -280,7 +280,7 @@ object TypeCoercion {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19889
**[Test build #84459 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84459/testReport)**
for PR 19889 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154863115
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -280,7 +280,7 @@ object TypeCoercion {
*/
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154863020
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -470,7 +470,7 @@ case class DataSource(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154863033
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1073,11 +1076,13 @@ class Analyzer(
* The
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154862947
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ---
@@ -241,7 +241,7 @@ class PlannerSuite extends SharedSQLContext {
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/19889
[SPARK-22690][ML] Imputer inherit HasOutputCols
## What changes were proposed in this pull request?
make `Imputer` inherit `HasOutputCols`
## How was this patch tested?
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154863003
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -666,7 +667,9 @@ class Analyzer(
* Generate a
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r154862787
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154862662
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19873
From the PR description, I am unable to tell the changes made in this PR.
We need a better description to explain what is the solution proposed in this
PR.
---
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/19840
@vanzin
according to @ueshin 's explanation, `PYSPARK_DRIVER_PYTHON` is only for
driver, if executor follows the order of
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154860853
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19874
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19888
**[Test build #84458 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84458/testReport)**
for PR 19888 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/19793#discussion_r154859418
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/deploy/rest/mesos/MesosRestServer.scala
---
@@ -77,10 +77,16 @@ private[mesos] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154859083
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1098,7 +1103,8 @@ class Analyzer(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154858997
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -666,7 +667,9 @@ class Analyzer(
*
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154859044
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1073,11 +1076,13 @@ class Analyzer(
* The
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154858146
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -470,7 +470,7 @@ case class DataSource(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154857858
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -666,7 +667,9 @@ class Analyzer(
*
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154857587
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -280,7 +280,7 @@ object TypeCoercion {
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154857148
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19874
**[Test build #84457 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84457/testReport)**
for PR 19874 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19874
BTW, I did not change all the rules in `object TypeCoercion` to propagate
the types, because not all of them need to propagate the types.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19874#discussion_r154855820
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/DecimalPrecision.scala
---
@@ -58,7 +58,7 @@ import
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154855703
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19873
LGTM, also cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154855592
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ---
@@ -241,7 +241,7 @@ class PlannerSuite extends SharedSQLContext {
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r15480
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19874
**[Test build #84456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84456/testReport)**
for PR 19874 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154855150
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
---
@@ -50,41 +50,6 @@ abstract class LogicalPlan
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19874#discussion_r154854338
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/TPCDSQuerySuite.scala ---
@@ -39,6 +40,8 @@ class TPCDSQuerySuite extends QueryTest with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19874#discussion_r154854356
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -850,3 +834,45 @@ object TypeCoercion {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r154854328
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154854071
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154854042
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19874#discussion_r154853933
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/TPCDSQuerySuite.scala ---
@@ -39,6 +40,8 @@ class TPCDSQuerySuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19885
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19885
**[Test build #84455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84455/testReport)**
for PR 19885 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19885
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84455/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154853724
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19874#discussion_r154853773
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -850,3 +834,45 @@ object TypeCoercion {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19877
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84452/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19877
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19877
**[Test build #84452 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84452/testReport)**
for PR 19877 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19852#discussion_r154852742
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -319,7 +319,7 @@ private[spark] abstract class BasePythonRunner[IN,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154852342
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154852231
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends Logging
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154852193
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends Logging
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154852162
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19885
**[Test build #84455 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84455/testReport)**
for PR 19885 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19888#discussion_r154851944
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -206,7 +206,8 @@ case class DropTableCommand(
try {
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19885
ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154850738
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/17702
gental ping @zsxwing
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19885
Is this assumption based on the implementation of Hadoop `FileSystem`? I
was thinking that wasb is an exception, for other we still keep the original
code.
@steveloughran would you
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19713
Hi, All.
It seems we missed `DROP TABLE IF EXISTS` cases here.
(I also forgot to check that during 2.2-backporting of this PR.)
This PR swallows `NoSuchTableException` and emits
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154848509
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user sadikovi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19810#discussion_r154848470
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -193,38 +195,68 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154848428
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154848336
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19888
**[Test build #84454 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84454/testReport)**
for PR 19888 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154848277
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154848218
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/19888
[SPARK-22686][SQL] DROP TABLE IF EXISTS should not throw AnalysisException
## What changes were proposed in this pull request?
During
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154848018
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154847802
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r154847703
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala
---
@@ -114,4 +114,194 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154847064
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -568,8 +574,13 @@ object DataSource extends Logging
Github user merlintang commented on the issue:
https://github.com/apache/spark/pull/19885
@jerryshao yes, hdfs://us...@nn1.com:8020 and hdfs://us...@nn1.com:8020
would consider as two filesystem, since the authority information should be
taken into consideration. that is why need to
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19878
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19717#discussion_r154846305
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -388,6 +388,32 @@ class SparkSubmitSuite
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19878
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19878#discussion_r154845901
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/hash.scala
---
@@ -730,23 +776,29 @@ case class HiveHash(children:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19869
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19869
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user CodingCat commented on the issue:
https://github.com/apache/spark/pull/19810
@sadikovi thanks for the review, I replied in comments
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user CodingCat commented on a diff in the pull request:
https://github.com/apache/spark/pull/19810#discussion_r154845669
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -193,38 +195,68 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19746
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84451/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19746
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19746
**[Test build #84451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84451/testReport)**
for PR 19746 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19717
**[Test build #84453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84453/testReport)**
for PR 19717 at commit
Github user CodingCat commented on a diff in the pull request:
https://github.com/apache/spark/pull/19810#discussion_r154844579
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala
---
@@ -52,6 +52,68 @@ object InMemoryRelation {
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19882#discussion_r154843730
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcQuerySuite.scala
---
@@ -614,11 +531,63 @@ class OrcQuerySuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19717
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19717
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84450/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19717
**[Test build #84450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84450/testReport)**
for PR 19717 at commit
1 - 100 of 490 matches
Mail list logo