Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69712/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16128
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16068
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16068
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69715/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16068
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16128
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69714/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16068#discussion_r91027951
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveUDFSuite.scala
---
@@ -487,6 +488,52 @@ class HiveUDFSuite extends QueryTest
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91023297
--- Diff: examples/src/main/r/ml/lda.R ---
@@ -0,0 +1,46 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16148
hmm, yea, I suspect these example can be easily forgotten while changes are
made - they could end up getting out of sync easily
---
If your project is set up for it, you can reply to this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16117
**[Test build #69718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69718/consoleFull)**
for PR 16117 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16014
**[Test build #69712 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69712/consoleFull)**
for PR 16014 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16161
can you highlight your approach to fix this bug? It looks to me that this
is not a general fix, e.g. `mutable.Map` will still fail.
---
If your project is set up for it, you can reply to this
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91024114
--- Diff: examples/src/main/r/ml/randomForest.R ---
@@ -0,0 +1,63 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91022864
--- Diff: examples/src/main/r/ml/isoreg.R ---
@@ -0,0 +1,42 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16148
**[Test build #69717 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69717/consoleFull)**
for PR 16148 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91022027
--- Diff: docs/sparkr.md ---
@@ -512,39 +512,33 @@ head(teenagers)
# Machine Learning
-SparkR supports the following machine learning
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91005161
--- Diff: docs/ml-classification-regression.md ---
@@ -75,6 +75,13 @@ More details on parameters can be found in the [Python
API documentation](api/py
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91024054
--- Diff: examples/src/main/r/ml/ml.R ---
@@ -0,0 +1,64 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91024237
--- Diff: docs/sparkr.md ---
@@ -512,39 +512,33 @@ head(teenagers)
# Machine Learning
-SparkR supports the following machine learning
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91022054
--- Diff: docs/sparkr.md ---
@@ -512,39 +512,33 @@ head(teenagers)
# Machine Learning
-SparkR supports the following machine learning
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91022606
--- Diff: examples/src/main/r/ml/gaussianMixture.R ---
@@ -0,0 +1,42 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91023264
--- Diff: examples/src/main/r/ml/lda.R ---
@@ -0,0 +1,46 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16148#discussion_r91005531
--- Diff: docs/ml-classification-regression.md ---
@@ -847,6 +893,14 @@ Refer to the [`IsotonicRegression` Python
docs](api/python/pyspark.ml.html#pyspa
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16148
@felixcheung That's a good question. We don't run all MLlib examples for
testing until [SPARK-12347](https://issues.apache.org/jira/browse/SPARK-12347)
was resolved.
---
If your project is set
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16014
thanks, updated. @rxin could you please review release-build.sh?
---
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/16148
**[Test build #69717 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69717/consoleFull)**
for PR 16148 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16148
Yeah, I have the same concerns, so we should push forward SPARK-12347 and
it should not be very hard to do that.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16148
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69717/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16148
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/16099
**[Test build #69719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69719/consoleFull)**
for PR 16099 at commit
Github user AnthonyTruchet commented on a diff in the pull request:
https://github.com/apache/spark/pull/16037#discussion_r91036283
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/optimization/LBFGS.scala ---
@@ -241,16 +241,27 @@ object LBFGS extends Logging {
Github user AnthonyTruchet commented on the issue:
https://github.com/apache/spark/pull/16037
@sethah Agreed, that's why I uggested to add a dedicated treeAggregate
wrapper to MLlin Utils which would take care of that without fiddling with
sparsity for each seqOP and comboOp. See
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16099
**[Test build #69720 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69720/consoleFull)**
for PR 16099 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16148
**[Test build #69721 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69721/consoleFull)**
for PR 16148 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13909
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69723/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13909
**[Test build #69723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69723/consoleFull)**
for PR 13909 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13909
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16099
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69720/
Test PASSed.
---
Github user lichenglin commented on the issue:
https://github.com/apache/spark/pull/15819
I'm using spark 2.0.2 I got a really big hive-stage folder.
May I delete the folder Manually ?
does it make any bad affect on warehouse?
---
If your project is set up for it, you can
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15467
@andrewor14 would it be possible to review this code? Or, could you please
create the similar PR from https://github.com/apache/spark/pull/13899?
Your original refactoring was great. In addition
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15468
@andrewor14 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 does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16099
**[Test build #69719 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69719/consoleFull)**
for PR 16099 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16099
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/16099
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69719/
Test PASSed.
---
Github user omuravskiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16006#discussion_r91035292
--- Diff:
external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/DirectKafkaInputDStream.scala
---
@@ -67,6 +67,10 @@ private[spark]
Github user omuravskiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16006#discussion_r91038777
--- Diff:
external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/DirectKafkaInputDStream.scala
---
@@ -143,9 +147,14 @@ private[spark]
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16117
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69718/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16117
**[Test build #69718 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69718/consoleFull)**
for PR 16117 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16117
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 kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r91039919
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -56,28 +56,49 @@ case class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r91039873
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvalHelper.scala
---
@@ -59,12 +59,36 @@ trait ExpressionEvalHelper
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13909
**[Test build #69723 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69723/consoleFull)**
for PR 13909 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r91039898
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -56,28 +56,49 @@ case class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r91039854
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -132,50 +153,80 @@ case class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13909#discussion_r91039840
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -132,50 +153,80 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16037
**[Test build #69722 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69722/consoleFull)**
for PR 16037 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16099
**[Test build #69720 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69720/consoleFull)**
for PR 16099 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16148
**[Test build #69721 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69721/consoleFull)**
for PR 16148 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16148
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69721/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16148
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/16099
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 jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/16168
[SPARK-18209][SQL] More robust view canonicalization without full SQL
expansion
## What changes were proposed in this pull request?
Currently we canonicalize the view definition to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16168
**[Test build #69724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69724/consoleFull)**
for PR 16168 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16139#discussion_r91047347
--- Diff: docs/ml-advanced.md ---
@@ -59,17 +59,22 @@ Given $n$ weighted observations $(w_i, a_i, b_i)$:
The number of features for each
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16139#discussion_r91047063
--- Diff: docs/ml-advanced.md ---
@@ -59,17 +59,22 @@ Given $n$ weighted observations $(w_i, a_i, b_i)$:
The number of features for each
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16139#discussion_r91046684
--- Diff: docs/ml-advanced.md ---
@@ -59,17 +59,22 @@ Given $n$ weighted observations $(w_i, a_i, b_i)$:
The number of features for each
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16037
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 yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16139#discussion_r91049472
--- Diff: docs/ml-advanced.md ---
@@ -59,17 +59,22 @@ Given $n$ weighted observations $(w_i, a_i, b_i)$:
The number of features for each
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16037
**[Test build #69722 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69722/consoleFull)**
for PR 16037 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16037
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69722/
Test PASSed.
---
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/16169
[SPARK-18326][SPARKR][ML] Review SparkR ML wrappers API for 2.1
## What changes were proposed in this pull request?
Reviewing SparkR ML wrappers API for 2.1 release, mainly issues include:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16169
**[Test build #69725 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69725/consoleFull)**
for PR 16169 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16168
cc @rxin @srinathshankar @nsyca
---
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/16099
**[Test build #69727 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69727/consoleFull)**
for PR 16099 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91053671
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -509,32 +509,42 @@ class Analyzer(
* Replaces
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91078953
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -151,6 +151,7 @@ case class CatalogTable(
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91053117
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -151,6 +151,7 @@ case class CatalogTable(
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91053897
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -509,32 +509,42 @@ class Analyzer(
* Replaces
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16168
also cc @gatorsmile
---
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91064615
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -126,6 +146,55 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91054484
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91051441
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -207,31 +205,56 @@ case class CreateViewCommand(
}
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91053965
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -509,32 +509,42 @@ class Analyzer(
* Replaces
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91051621
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -207,31 +205,56 @@ case class CreateViewCommand(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16171
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69729/
Test FAILed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16170
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16168
**[Test build #69724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69724/consoleFull)**
for PR 16168 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69724/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16171
**[Test build #69729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69729/consoleFull)**
for PR 16171 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16171
**[Test build #69729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69729/consoleFull)**
for PR 16171 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16171
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91064795
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91064962
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91065717
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91064259
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -126,6 +146,55 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91078407
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -114,9 +117,26 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91066027
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -126,6 +146,55 @@ private[hive] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91052968
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -207,31 +205,56 @@ case class CreateViewCommand(
}
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91051675
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -275,20 +297,26 @@ case class AlterViewAsCommand(
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16168#discussion_r91051761
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -275,20 +297,26 @@ case class AlterViewAsCommand(
401 - 500 of 596 matches
Mail list logo
