[GitHub] spark issue #14546: [SPARK-16955][SQL] Using ordinals in ORDER BY and GROUP ...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14546 **[Test build #63631 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63631/consoleFull)** for PR 14546 at commit

[GitHub] spark issue #14558: [SPARK-16508][SparkR] Fix warnings on undocumented/dupli...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14558 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] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14559 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63675/ Test PASSed. ---

[GitHub] spark issue #14397: [SPARK-16771][SQL] WITH clause should not fall into infi...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14397 **[Test build #63637 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63637/consoleFull)** for PR 14397 at commit

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution in CTE by ...

2016-08-12 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/14452 @dilipbiswal yea. Only when the execution result of CommonSubquery is asked, it will be materialized then. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution by dedupli...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14452 **[Test build #63665 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63665/consoleFull)** for PR 14452 at commit

[GitHub] spark issue #14609: [MINOR][Core] fix warnings on depreciated methods in Mes...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14609 **[Test build #63659 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63659/consoleFull)** for PR 14609 at commit

[GitHub] spark issue #14586: [SPARK-17003] [BUILD] [BRANCH-1.6] release-build.sh is m...

2016-08-12 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14586 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #14607: [SPARK-16905] SQL DDL: MSCK REPAIR TABLE (follow-up)

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14607 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] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14559 **[Test build #63675 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63675/consoleFull)** for PR 14559 at commit

[GitHub] spark issue #14596: [SPARK-12370] [Documentation] [Documentation should link...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14596 **[Test build #63682 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63682/consoleFull)** for PR 14596 at commit

[GitHub] spark pull request #14593: [MINOR][DOC] Fix style in examples across documen...

2016-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14593 --- 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] spark issue #14616: [SPARK-16955][SQL] Fix analysis error when using ordinal...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14616 **[Test build #63670 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63670/consoleFull)** for PR 14616 at commit

[GitHub] spark issue #14618: [SPARK-17030] [SQL] Remove/Cleanup HiveMetastoreCatalog....

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14618 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63676/ Test FAILed. ---

[GitHub] spark issue #14244: [SPARK-16598] [SQL] [TEST] Added a test case for verifyi...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14244 merged to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #14616: [SPARK-16955][SQL] Fix analysis error when using ordinal...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14616 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63670/ Test PASSed. ---

[GitHub] spark issue #14616: [SPARK-16955][SQL] Fix analysis error when using ordinal...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14616 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] spark issue #14596: [SPARK-12370] [Documentation] [Documentation should link...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14596 Jenkins add to whitelist --- 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] spark pull request #14244: [SPARK-16598] [SQL] [TEST] Added a test case for ...

2016-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14244 --- 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] spark issue #14596: [SPARK-12370] [Documentation] [Documentation should link...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14596 Great, that's all the occurrences, and I like that you added use of the SPARK_GITHUB_URL property. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #14593: [MINOR][DOC] Fix style in examples across documentation

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14593 Merged to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #14574: [SPARK-16985] Change dataFormat from yyyyMMddHHmm to yyy...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14574 Merged to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #14618: [SPARK-17030] [SQL] Remove/Cleanup HiveMetastoreCatalog....

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14618 **[Test build #63676 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63676/consoleFull)** for PR 14618 at commit

[GitHub] spark pull request #14618: [SPARK-17030] [SQL] Remove/Cleanup HiveMetastoreC...

2016-08-12 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14618#discussion_r74552290 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala --- @@ -1,494 +0,0 @@ -/* - * Licensed to the Apache

[GitHub] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14559 **[Test build #63675 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63675/consoleFull)** for PR 14559 at commit

[GitHub] spark issue #13893: [SPARK-14172][SQL] Hive table partition predicate not pa...

2016-08-12 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/13893 @cloud-fan I've send a PR to add `Scanner` operator in #14619 , please have a look at it when you have time, thanks! --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request #14616: [SPARK-16955][SQL] Fix analysis error when using ...

2016-08-12 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14616#discussion_r74556830 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -2223,3 +2223,29 @@ object TimeWindowing extends

[GitHub] spark pull request #14618: [SPARK-17030] [SQL] Remove/Cleanup HiveMetastoreC...

2016-08-12 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14618#discussion_r74550719 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala --- @@ -1,494 +0,0 @@ -/* - * Licensed to the Apache

[GitHub] spark issue #14229: [SPARK-16447][ML][SparkR] LDA wrapper in SparkR

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14229 **[Test build #63661 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63661/consoleFull)** for PR 14229 at commit

[GitHub] spark issue #14182: [SPARK-16444][SparkR]: Isotonic Regression wrapper in Sp...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14182 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63663/ Test PASSed. ---

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution by dedupli...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14452 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63658/ Test PASSed. ---

[GitHub] spark issue #14618: [SPARK-17030] [SQL] Remove/Cleanup HiveMetastoreCatalog....

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14618 **[Test build #63672 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63672/consoleFull)** for PR 14618 at commit

[GitHub] spark pull request #14615: [SPARK-17029] make toJSON not go through rdd form...

2016-08-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/14615#discussion_r74547698 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala --- @@ -155,13 +155,13 @@ class JsonFileFormat extends

[GitHub] spark pull request #14615: [SPARK-17029] make toJSON not go through rdd form...

2016-08-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/14615#discussion_r7454 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -2494,16 +2494,18 @@ class Dataset[T] private[sql]( * @since 2.0.0

[GitHub] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14559 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] spark issue #14229: [SPARK-16447][ML][SparkR] LDA wrapper in SparkR

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14229 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] spark issue #14571: [SPARK-16983][SQL] Add `prettyName` for row_number, dens...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14571 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] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution by dedupli...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14452 **[Test build #63658 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63658/consoleFull)** for PR 14452 at commit

[GitHub] spark issue #14616: [SPARK-16955][SQL] Fix analysis error when using ordinal...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14616 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63655/ Test PASSed. ---

[GitHub] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14559 **[Test build #63662 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63662/consoleFull)** for PR 14559 at commit

[GitHub] spark issue #14182: [SPARK-16444][SparkR]: Isotonic Regression wrapper in Sp...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14182 **[Test build #63663 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63663/consoleFull)** for PR 14182 at commit

[GitHub] spark pull request #14559: [SPARK-16968]Add additional options in jdbc when ...

2016-08-12 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/14559#discussion_r74542181 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala --- @@ -20,14 +20,21 @@ package

[GitHub] spark issue #14229: [SPARK-16447][ML][SparkR] LDA wrapper in SparkR

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14229 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63661/ Test PASSed. ---

[GitHub] spark pull request #14229: [SPARK-16447][ML][SparkR] LDA wrapper in SparkR

2016-08-12 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/14229#discussion_r74542484 --- Diff: mllib/src/main/scala/org/apache/spark/ml/r/LDAWrapper.scala --- @@ -0,0 +1,210 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark issue #14609: [MINOR][Core] fix warnings on depreciated methods in Mes...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14609 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] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution in CTE by ...

2016-08-12 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/14452 @gatorsmile Your second query example have two different CTE subqueries. And no one is reused in the query plan. So we will not see CommonSubquery. The optimized plan of this query looks like:

[GitHub] spark issue #14617: [SPARK-17019][Core] Expose on-heap and off-heap memory u...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14617 **[Test build #63657 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63657/consoleFull)** for PR 14617 at commit

[GitHub] spark pull request #14599: [SPARK-17013][SQL] handle corner case for negativ...

2016-08-12 Thread cloud-fan
Github user cloud-fan closed the pull request at: https://github.com/apache/spark/pull/14599 --- 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] spark issue #14397: [SPARK-16771][SQL] WITH clause should not fall into infi...

2016-08-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/14397 Hi, @hvanhovell . Could you review this again? After the last time you've seen, only the testcase is moved into a new testsuite. --- If your project is set up for it, you can reply to

[GitHub] spark issue #14608: [SPARK-17013][SQL] Parse negative numeric literals

2016-08-12 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14608 adding a space LGTM. I think `UnaryMinus` is the only special case? Adding parentheses at the outside of each expression should work for all other cases. --- If your project is set up for it,

[GitHub] spark issue #14102: [SPARK-16434][SQL] Avoid per-record type dispatch in JSO...

2016-08-12 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14102 thanks,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

[GitHub] spark issue #14111: [SPARK-16456][SQL] Reuse the uncorrelated scalar subquer...

2016-08-12 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/14111 I think this is duplicate to the merged one #14548? --- 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] spark issue #14599: [SPARK-17013][SQL] handle corner case for negative integ...

2016-08-12 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14599 closing in favor of #14608 --- 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] spark issue #14609: [MINOR][Core] fix warnings on depreciated methods in Mes...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14609 **[Test build #63650 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63650/consoleFull)** for PR 14609 at commit

[GitHub] spark pull request #14384: [Spark-16443][SparkR] Alternating Least Squares (...

2016-08-12 Thread junyangq
Github user junyangq commented on a diff in the pull request: https://github.com/apache/spark/pull/14384#discussion_r74537542 --- Diff: R/pkg/R/mllib.R --- @@ -632,3 +642,159 @@ setMethod("predict", signature(object = "AFTSurvivalRegressionModel"), function(object,

[GitHub] spark issue #13409: [SPARK-15667][SQL]Throw exception if columns number of o...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13409 **[Test build #63653 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63653/consoleFull)** for PR 13409 at commit

[GitHub] spark issue #14558: [SPARK-16508][SparkR] Fix warnings on undocumented/dupli...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14558 **[Test build #63654 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63654/consoleFull)** for PR 14558 at commit

[GitHub] spark issue #14571: [SPARK-16983][SQL] Add `prettyName` for row_number, dens...

2016-08-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/14571 Hi, @rxin . The target suite will be `WindowQuerySuite`, I'll move all and remove that testsuite. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #14182: [SPARK-16444][SparkR]: Isotonic Regression wrapper in Sp...

2016-08-12 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/14182 I haven't had a chance to look at the PR, but the function name `isoreg`seems in line with https://stat.ethz.ch/R-manual/R-devel/library/stats/html/isoreg.html -- So I'd say `spark.isoreg` sounds

[GitHub] spark issue #13146: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow set pythonEx...

2016-08-12 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/13146 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] spark issue #14561: [SPARK-16972][CORE] Move DriverEndpoint out of CoarseGra...

2016-08-12 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/14561 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 so,

[GitHub] spark issue #13409: [SPARK-15667][SQL]Throw exception if columns number of o...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13409 **[Test build #63653 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63653/consoleFull)** for PR 13409 at commit

[GitHub] spark issue #13409: [SPARK-15667][SQL]Throw exception if columns number of o...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13409 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63653/ Test FAILed. ---

[GitHub] spark pull request #11157: [SPARK-11714][Mesos] Make Spark on Mesos honor po...

2016-08-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11157#discussion_r74517417 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala --- @@ -358,6 +376,109 @@ private[mesos] trait

[GitHub] spark issue #14558: [SPARK-16508][SparkR] Fix warnings on undocumented/dupli...

2016-08-12 Thread junyangq
Github user junyangq commented on the issue: https://github.com/apache/spark/pull/14558 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution in CTE by ...

2016-08-12 Thread dilipbiswal
Github user dilipbiswal commented on the issue: https://github.com/apache/spark/pull/14452 @viirya Are we driving the execution of the CommonSubquery while compiling the main query ? For example, if we are explaining the query, we should not be executing the CommonSubquery, right ?

[GitHub] spark issue #11746: [SPARK-13602][CORE] Add shutdown hook to DriverRunner to...

2016-08-12 Thread BryanCutler
Github user BryanCutler commented on the issue: https://github.com/apache/spark/pull/11746 Hi @vanzin, have any more comments on this? Do you think it can be merged? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #14605: [SPARK-17022][YARN]Handle potential deadlock in driver h...

2016-08-12 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/14605 Argh. I really dislike `askWithRetry`. This is just another reason why. LGTM, since this maintains the current behavior. Merging to master / 2.0. --- If your project is set up for it, you

[GitHub] spark issue #14610: [SPARK-17026][MLlib][Tests] remove identical values subs...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14610 Yes, I wrote that. It's what the test is testing, that they return the same value. I get it, we know the implementation right now is definitely going to pass that test always. And I don't see any

[GitHub] spark pull request #11157: [SPARK-11714][Mesos] Make Spark on Mesos honor po...

2016-08-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11157#discussion_r74517443 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala --- @@ -358,6 +376,109 @@ private[mesos] trait

[GitHub] spark pull request #11746: [SPARK-13602][CORE] Add shutdown hook to DriverRu...

2016-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11746 --- 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] spark issue #14558: [SPARK-16508][SparkR] Fix warnings on undocumented/dupli...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14558 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63651/ Test FAILed. ---

[GitHub] spark issue #14608: [SPARK-17013][SQL] Parse negative numeric literals

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14608 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] spark issue #14610: [SPARK-17026][MLlib][Tests] remove identical values subs...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14610 **[Test build #63640 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63640/consoleFull)** for PR 14610 at commit

[GitHub] spark pull request #11157: [SPARK-11714][Mesos] Make Spark on Mesos honor po...

2016-08-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11157#discussion_r74517366 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala --- @@ -358,6 +376,109 @@ private[mesos] trait

[GitHub] spark issue #14608: [SPARK-17013][SQL] Parse negative numeric literals

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14608 **[Test build #63634 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63634/consoleFull)** for PR 14608 at commit

[GitHub] spark issue #14610: [SPARK-17026][MLlib][Tests] remove identical values subs...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14610 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] spark issue #14607: [SPARK-16905] SQL DDL: MSCK REPAIR TABLE (follow-up)

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14607 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63633/ Test PASSed. ---

[GitHub] spark pull request #14610: [SPARK-17026][MLlib][Tests] remove identical valu...

2016-08-12 Thread keypointt
GitHub user keypointt opened a pull request: https://github.com/apache/spark/pull/14610 [SPARK-17026][MLlib][Tests] remove identical values substraction https://issues.apache.org/jira/browse/SPARK-17026 ## What changes were proposed in this pull request? I'm not

[GitHub] spark pull request #14397: [SPARK-16771][SQL] WITH clause should not fall in...

2016-08-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/14397#discussion_r74504911 --- Diff: sql/core/src/test/resources/sql-tests/inputs/with.sql --- @@ -0,0 +1,14 @@ +create temporary view t as select * from values 0, 1, 2 as

[GitHub] spark issue #14551: [SPARK-16961][CORE] Fixed off-by-one error that biased r...

2016-08-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14551 Jenkins add to whitelist --- 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] spark issue #14586: [SPARK-17003] [BUILD] [BRANCH-1.6] release-build.sh is m...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14586 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] spark pull request #14397: [SPARK-16771][SQL] WITH clause should not fall in...

2016-08-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/14397#discussion_r74503932 --- Diff: sql/core/src/test/resources/sql-tests/inputs/with.sql --- @@ -0,0 +1,14 @@ +create temporary view t as select * from values 0, 1, 2 as t(id);

[GitHub] spark issue #13146: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow set pythonEx...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13146 **[Test build #63618 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63618/consoleFull)** for PR 13146 at commit

[GitHub] spark issue #14598: [SPARK-17018][SQL] literals.sql for testing literal pars...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14598 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] spark issue #14616: [SPARK-16955][SQL] Fix analysis error when using ordinal...

2016-08-12 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14616 Can we just add a check in `ResolveAggregateFunctions`? If `groupByOrdinal` is enabled and aggregate expressions contains integral literals, we skip it. --- If your project is set up for it, you

[GitHub] spark issue #14609: [MINOR][Core] fix warnings on depreciated methods in Mes...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14609 **[Test build #63638 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63638/consoleFull)** for PR 14609 at commit

[GitHub] spark issue #14551: [SPARK-16961][CORE] Fixed off-by-one error that biased r...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14551 **[Test build #63630 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63630/consoleFull)** for PR 14551 at commit

[GitHub] spark pull request #14608: [SPARK-17013][SQL] Parse negative numeric literal...

2016-08-12 Thread petermaxlee
GitHub user petermaxlee opened a pull request: https://github.com/apache/spark/pull/14608 [SPARK-17013][SQL] Parse negative numeric literals ## What changes were proposed in this pull request? This patch updates the SQL parser to parse negative numeric literals as numeric

[GitHub] spark pull request #14609: [MINOR][Core] fix warnings on depreciated methods...

2016-08-12 Thread keypointt
GitHub user keypointt opened a pull request: https://github.com/apache/spark/pull/14609 [MINOR][Core] fix warnings on depreciated methods in MesosClusterSchedulerSuite and DiskBlockObjectWriterSuite ## What changes were proposed in this pull request? Fixed warnings below

[GitHub] spark pull request #14598: [SPARK-17018][SQL] literals.sql for testing liter...

2016-08-12 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/14598#discussion_r74506995 --- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out --- @@ -0,0 +1,148 @@ +-- Automatically generated by SQLQueryTestSuite

[GitHub] spark issue #14617: [SPARK-17019][Core] Expose on-heap and off-heap memory u...

2016-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14617 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] spark pull request #14447: [SPARK-16445][MLlib][SparkR] Multilayer Perceptro...

2016-08-12 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/14447#discussion_r74530185 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/MultilayerPerceptronClassifierExample.scala --- @@ -32,6 +32,7 @@ object

[GitHub] spark issue #14586: [SPARK-17003] [BUILD] [BRANCH-1.6] release-build.sh is m...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14586 **[Test build #3221 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3221/consoleFull)** for PR 14586 at commit

[GitHub] spark issue #14598: [SPARK-17018][SQL] literals.sql for testing literal pars...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14598 **[Test build #63625 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63625/consoleFull)** for PR 14598 at commit

[GitHub] spark pull request #14522: [Spark-16508][SparkR] Split docs for arrange and ...

2016-08-12 Thread junyangq
Github user junyangq commented on a diff in the pull request: https://github.com/apache/spark/pull/14522#discussion_r74536151 --- Diff: R/pkg/R/DataFrame.R --- @@ -2121,7 +2121,7 @@ setMethod("arrange", }) #' @rdname arrange -#' @name orderBy ---

[GitHub] spark issue #14586: [SPARK-17003] [BUILD] [BRANCH-1.6] release-build.sh is m...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14586 **[Test build #63629 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63629/consoleFull)** for PR 14586 at commit

[GitHub] spark issue #14559: [SPARK-16968]Add additional options in jdbc when creatin...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14559 **[Test build #63667 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63667/consoleFull)** for PR 14559 at commit

[GitHub] spark issue #14447: [SPARK-16445][MLlib][SparkR] Multilayer Perceptron Class...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14447 **[Test build #63673 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63673/consoleFull)** for PR 14447 at commit

[GitHub] spark issue #14571: [SPARK-16983][SQL] Add `prettyName` for row_number, dens...

2016-08-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14571 **[Test build #63660 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63660/consoleFull)** for PR 14571 at commit

<    1   2   3   4   5   6   >