[GitHub] spark pull request #15417: [SPARK-17851][SQL][TESTS] Make sure all test sqls...

2016-10-25 Thread jiangxb1987
Github user jiangxb1987 commented on a diff in the pull request: https://github.com/apache/spark/pull/15417#discussion_r84840666 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/FilterPushdownSuite.scala --- @@ -1016,6 +1016,8 @@ class FilterPushdownSu

[GitHub] spark issue #15566: [SPARK-18026][SQL] should not always lowercase partition...

2016-10-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/15566 thanks for the review, 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

[GitHub] spark pull request #15566: [SPARK-18026][SQL] should not always lowercase pa...

2016-10-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15566 --- 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] spark issue #15417: [SPARK-17851][SQL][TESTS] Make sure all test sqls in cat...

2016-10-25 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15417 Thanks for your work! I finished the first round of reviews. --- 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] spark issue #15024: [SPARK-17470][SQL] unify path for data source table and ...

2016-10-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/15024 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 #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/15319 Thank you @sameeragarwal ! I've updated these codes following your advice, please have a look at them when you have time. --- If your project is set up for it, you can reply to this email and h

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15319 **[Test build #67495 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67495/consoleFull)** for PR 15319 at commit [`905eaa1`](https://github.com/apache/spark/commit/9

[GitHub] spark issue #15024: [SPARK-17470][SQL] unify path for data source table and ...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15024 **[Test build #67496 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67496/consoleFull)** for PR 15024 at commit [`b004b6c`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15319 **[Test build #67497 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67497/consoleFull)** for PR 15319 at commit [`45308d5`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #15417: [SPARK-17851][SQL][TESTS] Make sure all test sqls in cat...

2016-10-25 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/15417 Thank you @gatorsmile ! I'll address your comment ASAP! --- 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] spark issue #15555: [Minor][ML] Refactor clustering summary.

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

[GitHub] spark issue #15555: [Minor][ML] Refactor clustering summary.

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/1 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] spark issue #15555: [Minor][ML] Refactor clustering summary.

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

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15596 **[Test build #67498 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67498/consoleFull)** for PR 15596 at commit [`82ebff4`](https://github.com/apache/spark/commit/8

[GitHub] spark issue #15172: [SPARK-13331] AES support for over-the-wire encryption

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15172 **[Test build #67499 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67499/consoleFull)** for PR 15172 at commit [`76136c5`](https://github.com/apache/spark/commit/7

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

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

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15596 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] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

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

[GitHub] spark issue #15556: [SPARK-18010][Core] Reduce work performed for building u...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15556 **[Test build #67500 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67500/consoleFull)** for PR 15556 at commit [`43ce952`](https://github.com/apache/spark/commit/4

[GitHub] spark pull request #15556: [SPARK-18010][Core] Reduce work performed for bui...

2016-10-25 Thread vijoshi
Github user vijoshi commented on a diff in the pull request: https://github.com/apache/spark/pull/15556#discussion_r84849414 --- Diff: core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala --- @@ -554,10 +559,15 @@ private[history] class FsHistoryProvider(conf

[GitHub] spark issue #15615: [SPARK-17693] [SQL] Fixed Insert Failure To Data Source ...

2016-10-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/15615 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 fea

[GitHub] spark issue #15606: [SPARK-18070][SQL] binary operator should not consider n...

2016-10-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/15606 @gatorsmile yup, the `DataType.sameType` has no override --- 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] spark pull request #12394: [SPARK-14634][ML] Add BisectingKMeansSummary

2016-10-25 Thread zhengruifeng
Github user zhengruifeng commented on a diff in the pull request: https://github.com/apache/spark/pull/12394#discussion_r84852205 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/BisectingKMeans.scala --- @@ -228,14 +252,22 @@ class BisectingKMeans @Since("2.0.0") (

[GitHub] spark issue #12394: [SPARK-14634][ML] Add BisectingKMeansSummary

2016-10-25 Thread zhengruifeng
Github user zhengruifeng commented on the issue: https://github.com/apache/spark/pull/12394 @jkbradley +1 Sounds good. --- 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 #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15596 **[Test build #67501 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67501/consoleFull)** for PR 15596 at commit [`44c64e0`](https://github.com/apache/spark/commit/4

[GitHub] spark pull request #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous li...

2016-10-25 Thread zhengruifeng
GitHub user zhengruifeng opened a pull request: https://github.com/apache/spark/pull/15619 [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in BisectingKMeans ## What changes were proposed in this pull request? As commented by @jkbradley in https://github.com/apache/spark/pul

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15619 **[Test build #67502 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67502/consoleFull)** for PR 15619 at commit [`81460d4`](https://github.com/apache/spark/commit/8

[GitHub] spark issue #15428: [SPARK-17219][ML] enhanced NaN value handling in Bucketi...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15428 **[Test build #67493 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67493/consoleFull)** for PR 15428 at commit [`5274d4a`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15428: [SPARK-17219][ML] enhanced NaN value handling in Bucketi...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15428 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] spark issue #15428: [SPARK-17219][ML] enhanced NaN value handling in Bucketi...

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

[GitHub] spark pull request #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous li...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/15619#discussion_r84853656 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/BisectingKMeans.scala --- @@ -264,7 +264,6 @@ class BisectingKMeans @Since("2.0.0") ( v

[GitHub] spark issue #15524: [SPARK-17984][YARN][Mesos][Deploy][WIP] Add support for ...

2016-10-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15524 This should be closed in favor of https://github.com/apache/spark/pull/15579 at least --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request #15611: [SPARK-16988] [Spark Shell] spark history server ...

2016-10-25 Thread hayashidac
GitHub user hayashidac reopened a pull request: https://github.com/apache/spark/pull/15611 [SPARK-16988] [Spark Shell] spark history server log needs to be fixed to show https url when ssl is enabled spark history server log needs to be fixed to show https url when ssl is enabled

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread hayashidac
Github user hayashidac commented on the issue: https://github.com/apache/spark/pull/15611 I applied comments. Please check again. --- 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 #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15611 Can one of the admins verify this patch? --- 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 feat

[GitHub] spark issue #15597: [SPARK-18063][SQL] Failed to infer constraints over mult...

2016-10-25 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/15597 @sameeragarwal I've addressed your comments! Thank you! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark issue #15262: [SPARK-17690][STREAMING][SQL] Add mini-dfs cluster based...

2016-10-25 Thread ScrapCodes
Github user ScrapCodes commented on the issue: https://github.com/apache/spark/pull/15262 This is just a testing artifact at the moment. At some point, I had these tests failing with spark nightly build. Something somewhere changed (which I did no pursue what it was) and they

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15596 **[Test build #67498 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67498/consoleFull)** for PR 15596 at commit [`82ebff4`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15596 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] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

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

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15619 **[Test build #67503 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67503/consoleFull)** for PR 15619 at commit [`03f6a47`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #15597: [SPARK-18063][SQL] Failed to infer constraints over mult...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15597 **[Test build #67504 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67504/consoleFull)** for PR 15597 at commit [`3f07493`](https://github.com/apache/spark/commit/3

[GitHub] spark pull request #15611: [SPARK-16988] [Spark Shell] spark history server ...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/15611#discussion_r84859771 --- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala --- @@ -147,7 +147,10 @@ private[spark] abstract class WebUI( } /** Return th

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread hayashidac
Github user hayashidac commented on the issue: https://github.com/apache/spark/pull/15611 sorry, I fixed 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

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread sarutak
Github user sarutak commented on the issue: https://github.com/apache/spark/pull/15611 Jenkins, 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 wish

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15319 **[Test build #67495 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67495/consoleFull)** for PR 15319 at commit [`905eaa1`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15611 **[Test build #67505 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67505/consoleFull)** for PR 15611 at commit [`6f26e30`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

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

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15319 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] spark issue #15024: [SPARK-17470][SQL] unify path for data source table and ...

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

[GitHub] spark pull request #13891: [SPARK-6685][MLLIB]Use DSYRK to compute AtA in AL...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13891#discussion_r84863062 --- Diff: project/MimaExcludes.scala --- @@ -864,6 +864,9 @@ object MimaExcludes { // [SPARK-12221] Add CPU time to metrics ProblemFilte

[GitHub] spark issue #15024: [SPARK-17470][SQL] unify path for data source table and ...

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

[GitHub] spark issue #15024: [SPARK-17470][SQL] unify path for data source table and ...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15024 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] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15319 **[Test build #67497 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67497/consoleFull)** for PR 15319 at commit [`45308d5`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15319 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] spark issue #15319: [SPARK-17733][SQL] InferFiltersFromConstraints rule neve...

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

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15619 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] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15619 **[Test build #67502 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67502/consoleFull)** for PR 15619 at commit [`81460d4`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

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

[GitHub] spark issue #15599: [SPARK-18022][SQL] java.lang.NullPointerException instea...

2016-10-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15599 @maver1ck @gatorsmile are you OK with this change, at least? I feel like I'm missing the additional change you have in mind. I think the purpose here is just to avoid the NPE during exception handlin

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread sarutak
Github user sarutak commented on the issue: https://github.com/apache/spark/pull/15611 While I I noticed that all of `spark.ui.ssl` in `SSLOptionsSuite.scala` should be `spark.ssl.ui`. It's very minor so you can fix it too within this PR if you would like to. --- If your project is

[GitHub] spark pull request #15620: [SPARK-18091] [SQL] Deep if expressions cause Gen...

2016-10-25 Thread kapilsingh5050
GitHub user kapilsingh5050 opened a pull request: https://github.com/apache/spark/pull/15620 [SPARK-18091] [SQL] Deep if expressions cause Generated SpecificUnsafeProjection code to exceed JVM code size limit ## What changes were proposed in this pull request? Fix for SPARK

[GitHub] spark pull request #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of W...

2016-10-25 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/15621 [SPARK-17748][FOLLOW-UP][ML] Reorg variables of WeightedLeastSquares. ## What changes were proposed in this pull request? Reorg some variables of ```WeightedLeastSquares```. ## How w

[GitHub] spark issue #15611: [SPARK-16988] [Spark Shell] spark history server log nee...

2016-10-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15611 Looking good, thank you. I also wouldn't mind fixing the "spark.ui.ssl" -> "spark.ssl.ui" issue here because it's a little bit related. --- If your project is set up for it, you can reply to this em

[GitHub] spark issue #15620: [SPARK-18091] [SQL] Deep if expressions cause Generated ...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15620 Can one of the admins verify this patch? --- 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 feat

[GitHub] spark issue #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of Weighted...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15621 **[Test build #67506 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67506/consoleFull)** for PR 15621 at commit [`99ced09`](https://github.com/apache/spark/commit/9

[GitHub] spark issue #15620: [SPARK-18091] [SQL] Deep if expressions cause Generated ...

2016-10-25 Thread kapilsingh5050
Github user kapilsingh5050 commented on the issue: https://github.com/apache/spark/pull/15620 @rxin @liancheng @sameeragarwal Please review --- 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 thi

[GitHub] spark pull request #15513: [SPARK-17963][SQL][Documentation] Add examples (e...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/15513#discussion_r84867371 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/xpath.scala --- @@ -95,8 +119,16 @@ case class XPathInt(xml: Expression,

[GitHub] spark pull request #15513: [SPARK-17963][SQL][Documentation] Add examples (e...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/15513#discussion_r84867463 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala --- @@ -25,7 +25,11 @@ import org.apache.spark.sql.types._

[GitHub] spark pull request #15513: [SPARK-17963][SQL][Documentation] Add examples (e...

2016-10-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/15513#discussion_r84867405 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/xpath.scala --- @@ -108,8 +140,16 @@ case class XPathLong(xml: Expression

[GitHub] spark issue #15513: [SPARK-17963][SQL][Documentation] Add examples (extend) ...

2016-10-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15513 For constants vs literals etc that sounds like a potentially important change. It sounds like it touches more than just these doc strings, and the purpose of this change is more about improving the f

[GitHub] spark issue #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of Weighted...

2016-10-25 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/15621 @sethah I wish you would not mind, I made some reorg for variables of ```WeightedLeastSquares```. Since we use ```bBar, bbBar, aBar, abBar, aaBar``` in standardized space always, I put the standa

[GitHub] spark issue #15172: [SPARK-13331] AES support for over-the-wire encryption

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15172 **[Test build #67499 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67499/consoleFull)** for PR 15172 at commit [`76136c5`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15172: [SPARK-13331] AES support for over-the-wire encryption

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15172 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] spark issue #15172: [SPARK-13331] AES support for over-the-wire encryption

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

[GitHub] spark pull request #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of W...

2016-10-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/15621#discussion_r84869328 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -280,7 +289,7 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15619 **[Test build #67503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67503/consoleFull)** for PR 15619 at commit [`03f6a47`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15618: [SPARK-14914][CORE] Fix Resource not closed after using,...

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

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15619 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] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

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

[GitHub] spark issue #15618: [SPARK-14914][CORE] Fix Resource not closed after using,...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15618 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] spark issue #15618: [SPARK-14914][CORE] Fix Resource not closed after using,...

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

[GitHub] spark issue #15413: [SPARK-17847][ML] Reduce shuffled data size of GaussianM...

2016-10-25 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/15413 cc @sethah @jkbradley --- 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 wishe

[GitHub] spark issue #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous line in B...

2016-10-25 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/15619 LGTM, merged into master. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enable

[GitHub] spark pull request #15619: [SPARK-14634][ML][FOLLOWUP] Delete superfluous li...

2016-10-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15619 --- 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] spark issue #15556: [SPARK-18010][Core] Reduce work performed for building u...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15556 **[Test build #67500 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67500/consoleFull)** for PR 15556 at commit [`43ce952`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15556: [SPARK-18010][Core] Reduce work performed for building u...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15556 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] spark issue #15556: [SPARK-18010][Core] Reduce work performed for building u...

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

[GitHub] spark issue #15555: [Minor][ML] Refactor clustering summary.

2016-10-25 Thread zhengruifeng
Github user zhengruifeng commented on the issue: https://github.com/apache/spark/pull/1 Looks good to me. --- 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 #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15596 **[Test build #67501 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67501/consoleFull)** for PR 15596 at commit [`44c64e0`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

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

[GitHub] spark issue #15596: [SPARK-18089][SQL] Remove CollectLimitExec

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15596 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] spark issue #15618: [SPARK-14914][CORE] Fix Resource not closed after using,...

2016-10-25 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15618 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] spark issue #15377: [SPARK-17802] Improved caller context logging.

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15377 **[Test build #3372 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3372/consoleFull)** for PR 15377 at commit [`5acefad`](https://github.com/apache/spark/commit/

[GitHub] spark pull request #15436: [SPARK-17875] [BUILD] Remove unneeded direct depe...

2016-10-25 Thread srowen
Github user srowen closed the pull request at: https://github.com/apache/spark/pull/15436 --- 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] spark issue #15618: [SPARK-14914][CORE] Fix Resource not closed after using,...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15618 **[Test build #67507 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67507/consoleFull)** for PR 15618 at commit [`f3713d1`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of Weighted...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15621 **[Test build #67506 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67506/consoleFull)** for PR 15621 at commit [`99ced09`](https://github.com/apache/spark/commit/

[GitHub] spark issue #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of Weighted...

2016-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15621 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] spark issue #15621: [SPARK-17748][FOLLOW-UP][ML] Reorg variables of Weighted...

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

[GitHub] spark issue #15610: [SPARK-18076][CORE][SQL] Fix default Locale used in Date...

2016-10-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15610 **[Test build #67508 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67508/consoleFull)** for PR 15610 at commit [`7a909c8`](https://github.com/apache/spark/commit/7

  1   2   3   4   5   6   >