Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-102214506
yes it should use it, thanks i will give a PR 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 evacchi commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-102085238
Is `FunctionRegistry` supposed to use this `CatalystConf` as well ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5806
---
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 enab
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100116896
updated /cc @marmbrus , please take a look
---
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 d
Github user scwf commented on a diff in the pull request:
https://github.com/apache/spark/pull/5806#discussion_r29919305
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -32,6 +34,7 @@ private[spark] object SQLConf {
val CODEGEN_ENABLED = "spark.sql
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100116482
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100116475
[Test build #32198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32198/consoleFull)
for PR 5806 at commit
[`cd51712`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100116484
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100098188
[Test build #32198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32198/consoleFull)
for PR 5806 at commit
[`cd51712`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100098140
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100098157
Merged build started.
---
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
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100097995
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100097546
[Test build #32194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32194/consoleFull)
for PR 5806 at commit
[`cd51712`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100097550
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100097549
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100096966
[Test build #32194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32194/consoleFull)
for PR 5806 at commit
[`cd51712`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100096853
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100096864
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095731
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095732
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095730
[Test build #32190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32190/consoleFull)
for PR 5806 at commit
[`d4b724f`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095374
[Test build #32190 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32190/consoleFull)
for PR 5806 at commit
[`d4b724f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095261
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-100095270
Merged build started.
---
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
Github user scwf commented on a diff in the pull request:
https://github.com/apache/spark/pull/5806#discussion_r29915656
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala ---
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software Found
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5806#discussion_r29907886
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -41,11 +44,17 @@ object SimpleAnalyzer extends Analyzer(E
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5806#discussion_r29907776
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala ---
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software F
Github user duncanfinney commented on a diff in the pull request:
https://github.com/apache/spark/pull/5806#discussion_r29823886
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -41,11 +44,17 @@ object SimpleAnalyzer extends Analyz
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99651353
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/320
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99651352
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99651341
[Test build #32036 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32036/consoleFull)
for PR 5806 at commit
[`269cf21`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99622577
[Test build #32036 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32036/consoleFull)
for PR 5806 at commit
[`269cf21`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99622424
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99622384
Merged build triggered.
---
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 ha
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-99621837
/cc @marmbrus can you help 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98431783
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98431784
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/316
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98431779
[Test build #31678 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31678/consoleFull)
for PR 5806 at commit
[`b73df6c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98429941
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98429937
[Test build #31677 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31677/consoleFull)
for PR 5806 at commit
[`2a56515`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98429942
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/316
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98423441
[Test build #31678 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31678/consoleFull)
for PR 5806 at commit
[`b73df6c`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98423410
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98423409
Merged build triggered.
---
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98422807
Merged build triggered.
---
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 ha
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98422822
[Test build #31677 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31677/consoleFull)
for PR 5806 at commit
[`2a56515`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98422810
Merged build started.
---
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
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98422691
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
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98036406
the mima test failure should caused by #4015
---
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 scwf commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98029090
@marmbrus since this PR changed catalog and analyzer interface, it will
failed mima tests. Should i exclude them or close this PR ?
---
If your project is set up for it, yo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98028025
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98028029
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/314
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98028021
[Test build #31494 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31494/consoleFull)
for PR 5806 at commit
[`2a56515`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98022581
[Test build #31494 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31494/consoleFull)
for PR 5806 at commit
[`2a56515`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98022468
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5806#issuecomment-98022451
Merged build triggered.
---
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 ha
56 matches
Mail list logo
