Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16826#discussion_r103073487
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/ExperimentalMethods.scala ---
@@ -46,4 +46,10 @@ class ExperimentalMethods private[sql]() {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16826#discussion_r103073400
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1178,4 +1181,36 @@ class SessionCatalog(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17027
---
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/17027
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 wishes so,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17063
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/17027#discussion_r103073282
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -175,19 +175,14 @@ class Dataset[T] private[sql](
}
@transi
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17063
LGTM, 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 wish
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17065
**[Test build #73466 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73466/testReport)**
for PR 17065 at commit
[`5fa69b3`](https://github.com/apache/spark/commit/5f
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17065
CC @ron8hu @wzhfy
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17060
**[Test build #73463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73463/testReport)**
for PR 17060 at commit
[`4837de8`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17060
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/17060
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73463/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r103073176
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -361,57 +343,52 @@ case c
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r103073163
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -297,6 +278,8 @@ case cla
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r103073122
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -258,27 +246,20 @@ case c
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r103073081
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -140,56 +129,56 @@ case c
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r103073076
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -95,15 +84,16 @@ case cla
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17065
**[Test build #73465 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73465/testReport)**
for PR 17065 at commit
[`04cc681`](https://github.com/apache/spark/commit/04
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/17065
[SPARK-17075][SQL][followup] fix some minor issues and clean up the code
## What changes were proposed in this pull request?
This fixes some code style issues, naming issues, some missing
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17063
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17063
cc @cloud-fan @yhuai @sameeragarwal
---
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
ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17063
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73462/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16592
Let me continue the work now
---
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/17063
**[Test build #73462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73462/testReport)**
for PR 17063 at commit
[`d4ac13c`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17064
**[Test build #73464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73464/testReport)**
for PR 17064 at commit
[`dd6d8ca`](https://github.com/apache/spark/commit/dd
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/17064
[SPARK-19736][SQL] refreshByPath should clear all cached plans with the
specified path
## What changes were proposed in this pull request?
`Catalog.refreshByPath` can refresh the cache entr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17060
**[Test build #73463 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73463/testReport)**
for PR 17060 at commit
[`4837de8`](https://github.com/apache/spark/commit/48
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14789
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14789
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73461/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14789
**[Test build #73461 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73461/consoleFull)**
for PR 14789 at commit
[`b1cac5c`](https://github.com/apache/spark/commit/
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103071356
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -241,7 +243,8 @@ class SQLQueryTestSuite extends QueryTest with
S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17056
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/17056
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73460/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17056
**[Test build #73460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73460/testReport)**
for PR 17056 at commit
[`8595305`](https://github.com/apache/spark/commit/8
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103071094
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -98,7 +98,9 @@ class SQLQueryTestSuite extends QueryTest with
SharedSQ
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103070909
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -98,7 +98,9 @@ class SQLQueryTestSuite extends QueryTest with
Sha
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103070856
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -241,7 +243,8 @@ class SQLQueryTestSuite extends QueryTest with
Shared
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17063
**[Test build #73462 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73462/testReport)**
for PR 17063 at commit
[`d4ac13c`](https://github.com/apache/spark/commit/d4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
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/16976
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73457/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16976
**[Test build #73457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73457/testReport)**
for PR 16976 at commit
[`b7d56c6`](https://github.com/apache/spark/commit/b
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/17063
[SPARK-19735][SQL] Remove HOLD_DDLTIME from Catalog APIs
### What changes were proposed in this pull request?
As explained in Hive JIRA https://issues.apache.org/jira/browse/HIVE-12224,
HOLD
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103070711
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -241,7 +243,8 @@ class SQLQueryTestSuite extends QueryTest with
S
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103070622
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -121,7 +123,7 @@ class SQLQueryTestSuite extends QueryTest with
S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17062
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/17062
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73459/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17062
**[Test build #73459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73459/testReport)**
for PR 17062 at commit
[`332475c`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73456/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
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/16976
**[Test build #73456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73456/testReport)**
for PR 16976 at commit
[`6187f6c`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17061
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/17061
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73458/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17061
**[Test build #73458 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73458/testReport)**
for PR 17061 at commit
[`60af17f`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73455/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
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/16976
**[Test build #73455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73455/testReport)**
for PR 16976 at commit
[`b4e6983`](https://github.com/apache/spark/commit/b
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17047#discussion_r103069680
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2574,13 +2575,30 @@ private[spark] object Utils extends Logging {
def red
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73454/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16976
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/16976
**[Test build #73454 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73454/testReport)**
for PR 16976 at commit
[`321d082`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14963
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73451/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14963
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/14963
**[Test build #73451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73451/testReport)**
for PR 14963 at commit
[`215b7b3`](https://github.com/apache/spark/commit/2
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17052
@zsxwing got it
---
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 jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15415#discussion_r103068598
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -0,0 +1,339 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15415#discussion_r103068619
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -0,0 +1,339 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16626
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73450/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16626
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 viirya commented on the issue:
https://github.com/apache/spark/pull/16944
Few small comments left. 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 wi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16626
**[Test build #73450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73450/testReport)**
for PR 16626 at commit
[`4bd04fc`](https://github.com/apache/spark/commit/4
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068578
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,192 @@
+/*
+ * Licensed to the Apache Soft
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17056
**[Test build #73460 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73460/testReport)**
for PR 17056 at commit
[`8595305`](https://github.com/apache/spark/commit/85
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14789
**[Test build #73461 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73461/consoleFull)**
for PR 14789 at commit
[`b1cac5c`](https://github.com/apache/spark/commit/b
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068501
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15415
I don't think we need to support the default prediction (for empty/null
inputs) now. I agree we could use an inputer or add something as an option
later on.
Will take a final look now
Github user datumbox commented on the issue:
https://github.com/apache/spark/pull/17059
@srowen Yes, the behaviour of the method remains the same. This patch
helped me get a measurable improvement on GC overhead in Spark 2.0, so I though
that it would be beneficial for others. Anyway
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068442
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/17056
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 s
Github user HyukjinKwon closed the pull request at:
https://github.com/apache/spark/pull/16824
---
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 i
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16824
Thanks @holdenk, let me close this for now.
@davies, please give me your opinion. If you think it is worth I will
reopen. If not, I will resolve the JIRA.
---
If your project is set up
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068403
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
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/16826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73448/
Test PASSed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068378
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73448 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73448/testReport)**
for PR 16826 at commit
[`437b0bc`](https://github.com/apache/spark/commit/4
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103068302
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -121,7 +123,7 @@ class SQLQueryTestSuite extends QueryTest with
Shared
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103068318
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -241,7 +243,8 @@ class SQLQueryTestSuite extends QueryTest with
Shared
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17060#discussion_r103068296
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
@@ -98,7 +98,9 @@ class SQLQueryTestSuite extends QueryTest with
SharedSQ
Github user datumbox commented on the issue:
https://github.com/apache/spark/pull/17059
jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068277
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17062
**[Test build #73459 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73459/testReport)**
for PR 17062 at commit
[`332475c`](https://github.com/apache/spark/commit/33
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103068259
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Soft
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17060
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/16826
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/16826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73447/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17060
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73449/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73447 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73447/testReport)**
for PR 16826 at commit
[`fd11ee2`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17060
**[Test build #73449 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73449/testReport)**
for PR 17060 at commit
[`5f2a6a8`](https://github.com/apache/spark/commit/5
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/17062
[SPARK-17495] [SQL] Support date, timestamp and interval types in Hive hash
## What changes were proposed in this pull request?
- Timestamp hashing is done as per
[TimestampWritable.ha
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/17062
ok to test
---
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
1 - 100 of 569 matches
Mail list logo