[GitHub] spark issue #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

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

[GitHub] spark issue #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

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

[GitHub] spark issue #15697: [SparkR][Test]:remove unnecessary suppressWarnings

2016-11-01 Thread wangmiao1981
Github user wangmiao1981 commented on the issue: https://github.com/apache/spark/pull/15697 @HyukjinKwon It seems that windows test still fails with the same error --- 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] spark issue #15541: [SPARK-17637][Scheduler]Packed scheduling for Spark task...

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

[GitHub] spark issue #15710: [SPARK-18025] Use commit protocol API in structured stre...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15710 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 #15710: [SPARK-18025] Use commit protocol API in structured stre...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15710 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 #15710: [SPARK-18025] Use commit protocol API in structured stre...

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

[GitHub] spark issue #15710: [SPARK-18025] Use commit protocol API in structured stre...

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

[GitHub] spark issue #15710: [SPARK-18025] Use commit protocol API in structured stre...

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

[GitHub] spark issue #15673: [SPARK-17992][SQL] Return all partitions from HiveShim w...

2016-11-01 Thread ericl
Github user ericl commented on the issue: https://github.com/apache/spark/pull/15673 @mallman shall we go ahead and revert that in this PR? It didn't help with debugging the flaky test much. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #15647: [SPARK-18088][ML] Various ChiSqSelector cleanups

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

[GitHub] spark issue #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15546 Ah, I see. It's feature freeze. Thank you for informing 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

[GitHub] spark issue #15567: [SPARK-14393][SQL] values generated by non-deterministic...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15567 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 #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15546 Not rc, but the branch cut. --- 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 #15567: [SPARK-14393][SQL] values generated by non-deterministic...

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

[GitHub] spark issue #15697: [SparkR][Test]:remove unnecessary suppressWarnings

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15697 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 #15697: [SparkR][Test]:remove unnecessary suppressWarnings

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

[GitHub] spark issue #15647: [SPARK-18088][ML] Various ChiSqSelector cleanups

2016-11-01 Thread jkbradley
Github user jkbradley commented on the issue: https://github.com/apache/spark/pull/15647 Any more comments, or shall I merge this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark issue #15647: [SPARK-18088][ML] Various ChiSqSelector cleanups

2016-11-01 Thread jkbradley
Github user jkbradley commented on the issue: https://github.com/apache/spark/pull/15647 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 #15697: [SparkR][Test]:remove unnecessary suppressWarnings

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

[GitHub] spark issue #15567: [SPARK-14393][SQL] values generated by non-deterministic...

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

[GitHub] spark issue #15705: [SPARK-18183] [SPARK-18184] Fix INSERT [INTO|OVERWRITE] ...

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

[GitHub] spark issue #15705: [SPARK-18183] [SPARK-18184] Fix INSERT [INTO|OVERWRITE] ...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15705 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 #15705: [SPARK-18183] [SPARK-18184] Fix INSERT [INTO|OVERWRITE] ...

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

[GitHub] spark issue #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15546 Oh, 2.1-rc1 today? I see. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] spark issue #15688: [SPARK-18173][SQL] data source tables should support tru...

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15688 Could we add all the positive and negative cases in `ExternalCatalogSuite`, like the other APIs. Then, we can easily know whether the behaviors of `InMemoryCatalog` and `HiveExternalCatalog` are

[GitHub] spark issue #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15546 @dongjoon-hyun we are going to cut the release branch today (or tomorrow) so it would be good to get this in asap. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request #15688: [SPARK-18173][SQL] data source tables should supp...

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15688#discussion_r86023753 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -471,11 +471,30 @@ class InMemoryCatalog(

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-11-01 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/15721 This issue also brings up a more general point of how sample weights should be tested. It seems there are some common rules that all algorithms that incorporate sample weights are thought to follow

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

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

[GitHub] spark issue #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15546 Sorry for delay, @rxin . I'm still working on, and will update in a few days. --- 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 #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-11-01 Thread sethah
GitHub user sethah opened a pull request: https://github.com/apache/spark/pull/15721 [SPARK-17772][ML][TEST] Add test functions for ML sample weights ## What changes were proposed in this pull request? More and more ML algos are accepting sample weights, and they have been

[GitHub] spark issue #14906: [SPARK-17350][SQL] Disable default use of KryoSerializer...

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

[GitHub] spark issue #14906: [SPARK-17350][SQL] Disable default use of KryoSerializer...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14906 LGTM if tests pass 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 enabled and wishes

[GitHub] spark issue #15546: [SPARK-17982][SQL][WIP] SQLBuilder should wrap the gener...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15546 @dongjoon-hyun any update on this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] spark pull request #15567: [SPARK-14393][SQL] values generated by non-determ...

2016-11-01 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/15567#discussion_r86019635 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala --- @@ -264,23 +264,43 @@ trait NonSQLExpression extends

[GitHub] spark pull request #15567: [SPARK-14393][SQL] values generated by non-determ...

2016-11-01 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/15567#discussion_r86018687 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala --- @@ -264,23 +264,43 @@ trait NonSQLExpression extends

[GitHub] spark issue #15704: [SPARK-17732][SQL] ALTER TABLE DROP PARTITION should sup...

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

[GitHub] spark pull request #15719: [SPARK-18114][HOTFIX] Fix line-too-long style err...

2016-11-01 Thread srowen
Github user srowen closed the pull request at: https://github.com/apache/spark/pull/15719 --- 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 #15697: [SparkR][Test]:remove unnecessary suppressWarnings

2016-11-01 Thread wangmiao1981
Github user wangmiao1981 commented on the issue: https://github.com/apache/spark/pull/15697 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

[GitHub] spark issue #15719: [SPARK-18114][HOTFIX] Fix line-too-long style error from...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15719 Can you close this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] spark issue #15704: [SPARK-17732][SQL] ALTER TABLE DROP PARTITION should sup...

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

[GitHub] spark issue #15697: [SparkR][Test]:remove unnecessary suppressWarnings

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

[GitHub] spark pull request #15704: [SPARK-17732][SQL] ALTER TABLE DROP PARTITION sho...

2016-11-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/15704#discussion_r86015221 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -418,27 +419,53 @@ case class

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15659 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 #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

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

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

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

[GitHub] spark issue #15718: [SPARK-16839][SQL] Simplify Struct creation code path

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

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-01 Thread rgbkrk
Github user rgbkrk commented on the issue: https://github.com/apache/spark/pull/15659 As someone who followed #8318 consistently for a very long time, I can't imagine this is only about it being at the end of a release cycle. This is a critical issue for Python users. ---

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15659 BTW it is at the end of a release cycle right now so people are probably focusing more on bugs or looking at critical issues. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request #15354: [SPARK-17764][SQL] Add `to_json` supporting to co...

2016-11-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15354 --- 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 #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15659 What's the risk of having this in 2.1 after the branch is cut today? I guess not very big since this is mostly additive? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #15354: [SPARK-17764][SQL] Add `to_json` supporting to convert n...

2016-11-01 Thread marmbrus
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/15354 Thanks, I'm going to merge this 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

[GitHub] spark issue #15719: [SPARK-18114][HOTFIX] Fix line-too-long style error from...

2016-11-01 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15719 Yep, sorry for this right in the middle of RC time. The change itself should be a perfectly fine bug fix. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #15719: [SPARK-18114][HOTFIX] Fix line-too-long style error from...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15719 Merging in branch-2.0. I'm waiting for this to cut rc2. --- 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] spark pull request #15688: [SPARK-18173][SQL] data source tables should supp...

2016-11-01 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/15688#discussion_r86007181 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -471,11 +471,30 @@ class InMemoryCatalog(

[GitHub] spark issue #15496: [SPARK-17950] [Python] Match SparseVector behavior with ...

2016-11-01 Thread itg-abby
Github user itg-abby commented on the issue: https://github.com/apache/spark/pull/15496 Sorry, I forgot to ping anyone after the last update. @holdenk how does this look? What next steps should we take to push this PR? --- If your project is set up for it, you can reply to this

[GitHub] spark pull request #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

2016-11-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15720 --- 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 pull request #15688: [SPARK-18173][SQL] data source tables should supp...

2016-11-01 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/15688#discussion_r86007168 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -471,11 +471,30 @@ class InMemoryCatalog(

[GitHub] spark issue #15698: [SPARK-18182] Expose ReplayListenerBus.read() overload w...

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

[GitHub] spark issue #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15720 Merging in 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 #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

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

[GitHub] spark issue #15698: [SPARK-18182] Expose ReplayListenerBus.read() overload w...

2016-11-01 Thread andrewor14
Github user andrewor14 commented on the issue: https://github.com/apache/spark/pull/15698 LGTM 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

[GitHub] spark pull request #15720: [SPARK-18167] Disable flaky SQLQuerySuite test

2016-11-01 Thread ericl
GitHub user ericl opened a pull request: https://github.com/apache/spark/pull/15720 [SPARK-18167] Disable flaky SQLQuerySuite test We now know it's a persistent environmental issue that is causing this test to sometimes fail. One hypothesis is that some configuration is leaked from

[GitHub] spark issue #15662: [SPARK-18141][SQL] Fix to quote column names in the pred...

2016-11-01 Thread sureshthalamati
Github user sureshthalamati commented on the issue: https://github.com/apache/spark/pull/15662 @rxin @gatorsmile --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark issue #15594: [SPARK-18061][SQL][Security] Spark Thriftserver needs to...

2016-11-01 Thread cmirash
Github user cmirash commented on the issue: https://github.com/apache/spark/pull/15594 @steveloughran @yhuai Will you be able to help review this PR? --- 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 #15719: [SPARK-18114][HOTFIX] Fix line-too-long style error from...

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

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

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

[GitHub] spark issue #15659: [SPARK-1267][SPARK-18129] Allow PySpark to be pip instal...

2016-11-01 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/15659 jenkins retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark pull request #15719: [SPARK-18114][HOTFIX] Fix line-too-long style err...

2016-11-01 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/15719 [SPARK-18114][HOTFIX] Fix line-too-long style error from backport of SPARK-18114 ## What changes were proposed in this pull request? Fix style error introduced in cherry-pick of

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15677 I finished the last pass. Thanks for your work! --- 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 #15702: [SPARK-18124] Observed delay based Event Time Watermarks

2016-11-01 Thread marmbrus
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/15702 Not a dumb question! You can certainly use processing time if those are the semantics you require. I do think there is a little bit of work we need to do to ensure determinism for these

[GitHub] spark issue #15703: [SPARK-18186] Migrate HiveUDAFFunction to TypedImperativ...

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

[GitHub] spark issue #15703: [SPARK-18186] Migrate HiveUDAFFunction to TypedImperativ...

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15703 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 #15671: [SPARK-14567][ML]Add instrumentation logs to ML training...

2016-11-01 Thread jkbradley
Github user jkbradley commented on the issue: https://github.com/apache/spark/pull/15671 I just made a subtask for the algs covered here. Can you please link this PR to SPARK-18206 in the title instead of the umbrella task? --- If your project is set up for it, you can reply to

[GitHub] spark issue #15703: [SPARK-18186] Migrate HiveUDAFFunction to TypedImperativ...

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

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r86000653 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -1200,7 +1333,7 @@ case class

[GitHub] spark issue #15710: [SPARK-18025] Use commit protocol API in structured stre...

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

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85998364 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -1055,8 +1161,12 @@ case class

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85999566 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -1200,7 +1333,7 @@ case class

[GitHub] spark issue #15710: [SPARK-18025] Use commit protocol API in structured stre...

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

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85999505 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -1200,7 +1333,7 @@ case class

[GitHub] spark issue #15643: [SPARK-18114][MESOS] Fix mesos cluster scheduler generat...

2016-11-01 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15643 Ugh, the line became just a bit too long. HOTFIXing it straight away ... --- 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] spark pull request #15677: [SPARK-17963][SQL][Documentation] Add examples (e...

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85998607 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -1177,7 +1305,12 @@ case class

[GitHub] spark issue #15643: [SPARK-18114][MESOS] Fix mesos cluster scheduler generat...

2016-11-01 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15643 Gah, let me go see what happened there and fix it @rxin. It looked clean enough to cherry pick but I must have missed something --- If your project is set up for it, you can reply to this email and

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85997612 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala --- @@ -407,9 +436,15 @@ case class

[GitHub] spark issue #15697: [SparkR][Test]:remove unnecessary suppressWarnings

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

[GitHub] spark issue #15697: [SparkR][Test]:remove unnecessary suppressWarnings

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15697 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 #15697: [SparkR][Test]:remove unnecessary suppressWarnings

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

[GitHub] spark pull request #15637: [SPARK-18000] [SQL] Aggregation function for comp...

2016-11-01 Thread srinathshankar
Github user srinathshankar commented on a diff in the pull request: https://github.com/apache/spark/pull/15637#discussion_r85993076 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/MapAggregate.scala --- @@ -0,0 +1,332 @@ +/* + *

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85996873 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala --- @@ -121,7 +121,7 @@ case class Like(left:

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85996783 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala --- @@ -68,7 +68,7 @@ trait

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85996472 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala --- @@ -410,7 +410,7 @@ object Equality { }

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

2016-11-01 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15677#discussion_r85995869 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/nullExpressions.scala --- @@ -168,7 +200,12 @@ case class Nvl2(expr1:

[GitHub] spark issue #15643: [SPARK-18114][MESOS] Fix mesos cluster scheduler generat...

2016-11-01 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15643 @srowen this broke the build in branch-2.0. --- 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 #15605: [WIP] [SPARK-18067] [SQL] SortMergeJoin adds shuffle if ...

2016-11-01 Thread tejasapatil
Github user tejasapatil commented on the issue: https://github.com/apache/spark/pull/15605 @hvanhovell @cloud-fan @gatorsmile @yhuai : Any opinions about the approach described in my previous comment ? --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #15718: [SPARK-16839][SQL] Simplify Struct creation code path

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

[GitHub] spark issue #15718: [SPARK-16839][SQL] Simplify Struct creation code path

2016-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15718 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 #15718: [SPARK-16839][SQL] Simplify Struct creation code path

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

[GitHub] spark issue #15705: [SPARK-18183] [SPARK-18184] Fix INSERT [INTO|OVERWRITE] ...

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

<    1   2   3   4   5   6   7   8   >