Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15975#discussion_r89684437
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -220,22 +228,14 @@ class DataFrameReader private[sql](sparkSession:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16003
**[Test build #69200 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69200/consoleFull)**
for PR 16003 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16018
**[Test build #69199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69199/consoleFull)**
for PR 16018 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16018#discussion_r89684361
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala
---
@@ -309,32 +309,17 @@ case class PreWriteCheck(conf:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16020
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69198/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16020
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/16020
**[Test build #69198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69198/consoleFull)**
for PR 16020 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15958
The `Non-Goals` in the PR description is also confusing. Basically, what
this PR does is to explicitly output an unsupported error message if users
specify PARTITION. Could you also update that?
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/15136
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15136
The behavior of Hive 2.1 is different from Hive 1.6. Hive 2.1 does not
invalidate the stats, but only mark it inaccurate. Thus, this PR can be closed.
Thanks!
---
If your project is set up for
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15958
@gatorsmile I've updated the title, thank you!
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16020
**[Test build #69198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69198/consoleFull)**
for PR 16020 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r89683595
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -574,7 +574,12 @@ object ScalaReflection extends
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/16020
[SPARK-18596][ML] add checking and caching to bisecting kmeans
## What changes were proposed in this pull request?
jira: https://issues.apache.org/jira/browse/SPARK-18596
This is a follow
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r89683544
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -643,8 +646,14 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r89683532
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -605,7 +608,7 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r89683520
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -590,7 +591,11 @@ object ScalaReflection extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15780#discussion_r89683485
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -590,7 +591,11 @@ object ScalaReflection extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15975#discussion_r89683214
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -220,22 +228,14 @@ class DataFrameReader private[sql](sparkSession:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16018
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16019
so the actual difference is, if the table exists and `ignoreIfExists` is
true, previously we will still try to create this table, but logically we
should do nothing.
Since we are
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15998
where is the speed-up come from? Is it because the hive API
`getPartitionNames` is faster than `getPartitions`? Or is it because we
generate the partition string(`a=1/b=2/c=3`) at hive side and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15255
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69197/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15255
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/15255
**[Test build #69197 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69197/consoleFull)**
for PR 15255 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15998
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69195/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15998
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/15998
**[Test build #69195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69195/consoleFull)**
for PR 15998 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89681535
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -730,6 +730,23 @@ class SessionCatalog(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16019
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69196/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16019
cc @cloud-fan @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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16019
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/16019
**[Test build #69196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69196/consoleFull)**
for PR 16019 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69194/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16018
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/16018
**[Test build #69194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69194/consoleFull)**
for PR 16018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15255
**[Test build #69197 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69197/consoleFull)**
for PR 15255 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15255
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/15958
Could you update the PR title? The current one is confusing. This PR is not
a bug fix.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89680470
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -922,6 +923,29 @@ private[spark] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16019
**[Test build #69196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69196/consoleFull)**
for PR 16019 at commit
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16019
[SPARK-18595] [SQL] Handling ignoreIfExists in HiveExternalCatalog
createTable API
### What changes were proposed in this pull request?
Currently, `HiveExternalCatalog`'s `createTable`
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16015
Thank you for merging, @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
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89679682
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -922,6 +923,29 @@ private[spark] class
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16015
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16015
Merging to master/2.1/2.0. 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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14801#discussion_r89679622
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -171,9 +172,13 @@ private[spark] class
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89679527
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -922,6 +923,29 @@ private[spark] class
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89679485
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -922,6 +923,29 @@ private[spark] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15998
**[Test build #69195 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69195/consoleFull)**
for PR 15998 at commit
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89679477
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -189,6 +189,21 @@ abstract class ExternalCatalog {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15975
Any more comment? : )
Also cc @cloud-fan @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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16018
cc @cloud-fan @yhuai
---
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/16018
**[Test build #69194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69194/consoleFull)**
for PR 16018 at commit
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16018
[SPARK-18594] [SQL] Name Validation of Databases/Tables
### What changes were proposed in this pull request?
Currently, the name validation checks are limited to table creation. It is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69193/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
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/16013
**[Test build #69193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69193/consoleFull)**
for PR 16013 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16015
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/16015
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69192/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16015
**[Test build #69192 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69192/consoleFull)**
for PR 16015 at commit
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89678628
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala
---
@@ -482,6 +482,19 @@ class InMemoryCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69191/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
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/16013
**[Test build #69191 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69191/consoleFull)**
for PR 16013 at commit
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89678259
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -922,6 +923,29 @@ private[spark] class
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/15998#discussion_r89678233
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala
---
@@ -482,6 +482,19 @@ class InMemoryCatalog(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16015
Thank you, @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
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16015
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69190/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
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/16013
**[Test build #69190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69190/consoleFull)**
for PR 16013 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69189/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16013
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/16013
**[Test build #69189 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69189/consoleFull)**
for PR 16013 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16013
@srowen, this PR become unintentionally bigger than I thought. I am happy
to split this into multiple ones if you think so. I left some comments on the
changes where the output images might be
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89677052
--- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala ---
@@ -191,7 +197,10 @@ object JdbcRDD {
* The RDD takes care of closing the
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/16014#discussion_r89677001
--- Diff: dev/create-release/release-build.sh ---
@@ -189,6 +189,9 @@ if [[ "$1" == "package" ]]; then
SHA512 $PYTHON_DIST_NAME > \
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16013
@srowen, this PR become unintentionally bigger than I thought. I am happy
to split this into multiple ones if you think so. I left some comments on the
changes where the output images might be
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16013
**[Test build #69193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69193/consoleFull)**
for PR 16013 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16015
**[Test build #69192 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69192/consoleFull)**
for PR 16015 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676820
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -165,32 +165,56 @@ object ParamValidators {
s" of unexpected
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676784
--- Diff:
core/src/main/scala/org/apache/spark/util/random/SamplingUtils.scala ---
@@ -67,20 +67,23 @@ private[spark] object SamplingUtils {
}
Github user weiqingy commented on the issue:
https://github.com/apache/spark/pull/15941
Thanks for review this, @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
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676738
--- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala ---
@@ -191,7 +197,10 @@ object JdbcRDD {
* The RDD takes care of closing the
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676728
--- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala ---
@@ -151,7 +154,10 @@ object JdbcRDD {
* The RDD takes care of closing the
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676716
--- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala ---
@@ -41,7 +41,10 @@ private[spark] class JdbcPartition(idx: Int, val lower:
Long,
Github user weiqingy closed the pull request at:
https://github.com/apache/spark/pull/15960
---
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 weiqingy commented on the issue:
https://github.com/apache/spark/pull/15960
Thanks @rxin @srowen @holdenk.
---
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676519
--- Diff:
core/src/main/scala/org/apache/spark/util/random/StratifiedSamplingUtils.scala
---
@@ -160,12 +161,20 @@ private[spark] object
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676506
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -1184,8 +1187,13 @@ abstract class RDD[T: ClassTag](
*
* @note This
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16013
**[Test build #69191 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69191/consoleFull)**
for PR 16013 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676461
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -57,9 +57,17 @@ import
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676437
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -216,10 +216,13 @@ abstract class BaseRelation {
/**
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676433
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -216,10 +216,13 @@ abstract class BaseRelation {
/**
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676419
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/JdbcDialects.scala ---
@@ -40,9 +40,14 @@ case class JdbcType(databaseTypeDefinition :
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676409
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1066,9 +1073,12 @@ object functions {
* within each partition in
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676398
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1048,9 +1052,12 @@ object functions {
* within each partition in
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676381
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -474,7 +474,9 @@ object functions {
/**
* Aggregate
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16013#discussion_r89676335
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala ---
@@ -44,7 +44,9 @@ final class DataFrameStatFunctions
1 - 100 of 189 matches
Mail list logo