Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63341/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63341/consoleFull)**
for PR 12819 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r73824598
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1441,6 +1441,12 @@ object PushPredicateThroughJoin
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14522#discussion_r73824558
--- Diff: R/pkg/R/generics.R ---
@@ -551,7 +551,7 @@ setGeneric("merge")
#' @export
setGeneric("mutate", function(.data, ...)
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13758
@hvanhovell , @cloud-fan , @davies would it be possible to review 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 project
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/11956
@davies, I hope that you have some bandwidth to review PRs. Could you
please review this, too?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14522#discussion_r73824520
--- Diff: R/pkg/R/WindowSpec.R ---
@@ -82,16 +82,18 @@ setMethod("partitionBy",
}
})
-#' orderBy
+#'
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14522#discussion_r73824487
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2121,7 +2121,7 @@ setMethod("arrange",
})
#' @rdname arrange
-#' @name orderBy
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14113
LGTM, pending jenkins
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14533
**[Test build #63344 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63344/consoleFull)**
for PR 14533 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14522#discussion_r73824170
--- Diff: R/pkg/R/WindowSpec.R ---
@@ -82,10 +82,18 @@ setMethod("partitionBy",
}
})
-#' @rdname orderBy
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/14533
[SPARK-16606] [CORE] isleading warning for SparkContext.getOrCreate "WARN
SparkContext: Use an existing SparkContext, some configuration may not take
effect."
## What changes were proposed in this
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14489#discussion_r73823955
--- Diff: R/pkg/R/DataFrame.R ---
@@ -411,7 +411,7 @@ setMethod("coltypes",
#'
#' Set the column types of a SparkDataFrame.
#'
-#'
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14113
**[Test build #63343 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63343/consoleFull)**
for PR 14113 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14507
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14532
**[Test build #63342 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63342/consoleFull)**
for PR 14532 at commit
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823481
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -396,6 +425,10 @@ class
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14507
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
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
A couple minor style issues then LGTM. Will ping the committer when style
issues are resolved.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823297
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14422
@xubo245 you need to close the issues, we can't do so directly.
---
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 mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823241
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823225
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -396,6 +425,10 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14532
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14532
Jenkins test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14532
Thank you, yeah was going to fix those. 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
GitHub user Sherry302 opened a pull request:
https://github.com/apache/spark/pull/14532
SPARK-16945: Fix Java Lint errors
## What changes were proposed in this pull request?
This PR is to fix the minor Java linter errors as following:
[ERROR]
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14530#discussion_r73823171
--- Diff:
core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala ---
@@ -77,6 +77,18 @@ class StorageStatusListener(conf: SparkConf)
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823074
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73823040
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user jisookim0513 commented on the issue:
https://github.com/apache/spark/pull/10212
@vanzin sure will do
---
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 mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73822922
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63341 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63341/consoleFull)**
for PR 12819 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14411
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14523
This is not a regression, but Spark 2.0 has the issue. 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14523
**[Test build #63340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63340/consoleFull)**
for PR 14523 at commit
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73822562
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -341,6 +344,32 @@ class
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14175#discussion_r73822546
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackendSuite.scala
---
@@ -396,6 +425,10 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63337/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13701
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63337/consoleFull)**
for PR 13701 at commit
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14511
retest please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13893#discussion_r73821493
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/patterns.scala
---
@@ -64,10 +64,17 @@ object PhysicalOperation extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #63339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63339/consoleFull)**
for PR 14531 at commit
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/14531
[SPARK-16943] [SPAR] [SQL] Fix multiple bugs in CREATE TABLE LIKE command
### What changes were proposed in this pull request?
### How was this patch tested?
Added a few test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63338/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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/12819
**[Test build #63338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63338/consoleFull)**
for PR 12819 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13886#discussion_r73820673
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -357,6 +357,10 @@ trait CheckAnalysis extends
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14410
@cloud-fan Sure, will do it soon. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14113
Will update it tonight. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63335/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63335/consoleFull)**
for PR 14530 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14518
IMHO, I would prefer `compression` over `orc.compress` because I believe we
should promote to use `compression` rather than `orc.compress` for consistency
with other datasources.
As
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14155
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63336/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14155
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14155
**[Test build #63336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63336/consoleFull)**
for PR 14155 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63338/consoleFull)**
for PR 12819 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14518
What if users set both `compression` and `orc.compress`? It looks to me
that `orc.compress` is for ORC only and should have higher priority over
`compression` in ORC data source.
cc
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13867
cc @hvanhovell
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14113
can you update it? thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13886#discussion_r73817954
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -357,6 +357,10 @@ trait CheckAnalysis extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14472#discussion_r73817418
--- Diff:
sql/core/src/test/resources/sql-tests/results/number-format.sql.out ---
@@ -0,0 +1,50 @@
+-- Automatically generated by
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14529
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63334/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13701
**[Test build #63337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63337/consoleFull)**
for PR 13701 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14529
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14529
**[Test build #63334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63334/consoleFull)**
for PR 14529 at commit
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/14218
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
enabled and wishes so, or if the
Github user nsyca commented on the issue:
https://github.com/apache/spark/pull/14411
Thanks, @gatorsmile. This time I ran `dev/lint-scala`. Hope it's my last
attempt to get this work thru.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14472#discussion_r73816313
--- Diff: dev/.rat-excludes ---
@@ -99,4 +99,5 @@ spark-deps-.*
.*tsv
org.apache.spark.scheduler.ExternalClusterManager
.*\.sql
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14528
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63332/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14528
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14528
**[Test build #63332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63332/consoleFull)**
for PR 14528 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14518
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/6/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14518
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14518
**[Test build #6 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/6/consoleFull)**
for PR 14518 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/14065
@vazin, looks like I missed that comment, I will address that today.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14518
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/14518
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63331/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14518
**[Test build #63331 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63331/consoleFull)**
for PR 14518 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14155
**[Test build #63336 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63336/consoleFull)**
for PR 14155 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63335/consoleFull)**
for PR 14530 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r73815242
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1218,11 +1151,6 @@ class DDLSuite extends QueryTest with
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r73815246
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1523,12 +1451,6 @@ class DDLSuite extends QueryTest with
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r73815233
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1109,11 +1047,6 @@ class DDLSuite extends QueryTest with
GitHub user SaintBacchus opened a pull request:
https://github.com/apache/spark/pull/14530
[SPARK-16868][Web Ui] Fix executor be both dead and alive on executor ui.
## What changes were proposed in this pull request?
In a heavy pressure of the spark application, since the
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14523
LGTM, is it a regression for 2.0?
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14523#discussion_r73814553
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/MetadataCacheSuite.scala ---
@@ -85,4 +86,28 @@ class MetadataCacheSuite extends QueryTest with
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14410
Hi @gatorsmile can you send out a PR first? We definitely need it for 2.1,
and we can decide if we should backport it to 2.0 later.
---
If your project is set up for it, you can reply to this
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14410
now I think it's easy to catch `saveAsTable` for hive tables:
`case CreateTable(tableDesc, SaveMode.APPEND, Some(query)) if
tableDesc.provider.get != "hive" =>`
We can add a new rule
Github user erfangc commented on a diff in the pull request:
https://github.com/apache/spark/pull/13818#discussion_r73814190
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -298,6 +298,7 @@ case class InsertIntoHiveTable(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14389
to refactor the type widen rules, i.e. we only need 2 rules:
`findWiderType` and `findWiderTypeAndPromoteToString`.
---
If your project is set up for it, you can reply to this email and have
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14523
cc @rxin @cloud-fan This is a simple fix. Could you review the fix? 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r73814104
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -25,55 +25,57 @@
import
Github user alicegugu commented on the issue:
https://github.com/apache/spark/pull/14218
@ericl thanks for the comments, comments incorporated in new commit
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14529
**[Test build #63334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63334/consoleFull)**
for PR 14529 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14529
BTW, `CSVRelation` also exists in a separate file but
https://github.com/apache/spark/pull/13988 includes this change.
---
If your project is set up for it, you can reply to this email and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13818#discussion_r73813847
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -298,6 +298,7 @@ case class InsertIntoHiveTable(
1 - 100 of 223 matches
Mail list logo