[GitHub] spark pull request: [SPARK-11105][yarn] Distribute log4j.propertie...

2015-10-15 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/9118#discussion_r42146188 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -340,6 +340,14 @@ private[spark] class Client( "for

[GitHub] spark pull request: [SPARK-11093] [Core] ChildFirstURLClassLoader#...

2015-10-15 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/9106#issuecomment-148450771 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

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148455862 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11051][Core] Do not allow local checkpo...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9072#issuecomment-148457932 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

[GitHub] spark pull request: [SPARK-11051][Core] Do not allow local checkpo...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9072#issuecomment-148457934 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11066] Update DAGScheduler's "misbehave...

2015-10-15 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9076#issuecomment-148458829 OK, 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

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148460264 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

[GitHub] spark pull request: [SPARK-10979] [SparkR] Sparkrmerge: Add merge ...

2015-10-15 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9012#issuecomment-148467105 @NarineK Could you also bring this up to date with master branch ? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/9114#discussion_r42148461 --- Diff: launcher/src/test/java/org/apache/spark/launcher/SparkSubmitCommandBuilderSuite.java --- @@ -80,7 +82,7 @@ public void testCliParser() throws

[GitHub] spark pull request: [SPARK-11051][Core] Do not allow local checkpo...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9072#issuecomment-148451543 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148455905 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148459073 [Test build #43792 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43792/consoleFull) for PR 8416 at commit

[GitHub] spark pull request: [SPARK][SPARK-10842]Eliminate creating duplica...

2015-10-15 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/8923#issuecomment-148445380 This looks good, but can you add a test case? Mostly I'd like to understand the dependency structure that leads to this (perhaps there are other corner cases related to

[GitHub] spark pull request: [SPARK-11093] [Core] ChildFirstURLClassLoader#...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9106 --- 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: [SPARK-11051][Core] Do not allow local checkpo...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9072#issuecomment-148451518 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9114 --- 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: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148453594 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148453660 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148457207 [Test build #43791 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43791/consoleFull) for PR 8512 at commit

[GitHub] spark pull request: fix typo bellow -> below

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9136#issuecomment-148461262 [Test build #1906 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1906/consoleFull) for PR 9136 at commit

[GitHub] spark pull request: [SPARK-10979] [SparkR] Sparkrmerge: Add merge ...

2015-10-15 Thread NarineK
Github user NarineK commented on the pull request: https://github.com/apache/spark/pull/9012#issuecomment-148465208 yes, I'll do that. Let me know if you have any more comments ... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: fix typo bellow -> below

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9136#issuecomment-148465446 [Test build #1906 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1906/console) for PR 9136 at commit

[GitHub] spark pull request: fix typo bellow -> below

2015-10-15 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9136#issuecomment-148466905 Unrelated failure, I'm going to merge soon anyway --- 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: [SPARK-11047] Internal accumulators miss the i...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9061 --- 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: [SPARK-11047] Internal accumulators miss the i...

2015-10-15 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9061#issuecomment-148468064 Thanks - I've merged 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

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148450238 LGTM, I'll fix the nit while merging. --- 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 pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148460269 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148460253 [Test build #43792 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43792/console) for PR 8416 at commit

[GitHub] spark pull request: [SPARK-11066] Update DAGScheduler's "misbehave...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9076 --- 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: [SPARK-11124] JsonParser/Generator should be c...

2015-10-15 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9130#discussion_r42154786 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2153,6 +2153,9 @@ private[spark] object Utils extends Logging {

[GitHub] spark pull request: [SPARK-11039][Documentation][Web UI] Document ...

2015-10-15 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9052#issuecomment-148501117 LGTM, so I'm going to merge this into master and branch-1.5. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-11039][Documentation][Web UI] Document ...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9052 --- 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: [SPARK-11124] JsonParser/Generator should be c...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9130#issuecomment-148290896 [Test build #43767 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43767/consoleFull) for PR 9130 at commit

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42157083 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala --- @@ -305,7 +305,7 @@ private[netty] class NettyRpcEnvFactory extends

[GitHub] spark pull request: [SPARK-11119][SQL] cleanup for unsafe array an...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9131#issuecomment-148473492 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

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread jodersky
Github user jodersky commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42157953 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala --- @@ -305,7 +305,7 @@ private[netty] class NettyRpcEnvFactory extends

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread jodersky
Github user jodersky commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148473495 Please hold --- 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: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/9123#discussion_r42158115 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -482,7 +482,9 @@ class Analyzer( val

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread jodersky
Github user jodersky commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42157349 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala --- @@ -305,7 +305,7 @@ private[netty] class NettyRpcEnvFactory extends

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148478062 [Test build #43797 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43797/consoleFull) for PR 8416 at commit

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148479569 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10412][SQL] report memory usage for tun...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8931#issuecomment-148482047 Build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-10412][SQL] report memory usage for tun...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8931#issuecomment-148482080 Build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-10412][SQL] report memory usage for tun...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8931#issuecomment-148483326 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11131] [core] Fix race in worker regist...

2015-10-15 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/9138 [SPARK-11131] [core] Fix race in worker registration protocol. Because the registration RPC was not really an RPC, but a bunch of disconnected messages, it was possible for other messages to be

[GitHub] spark pull request: [SPARK-11131] [core] Fix race in worker regist...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9138#issuecomment-148483325 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10186][SQL] Array types using JDBCRDD a...

2015-10-15 Thread mariusvniekerk
GitHub user mariusvniekerk opened a pull request: https://github.com/apache/spark/pull/9137 [SPARK-10186][SQL] Array types using JDBCRDD and postgres This change allows reading from jdbc array column types for the postgresql dialect. This also opens up some implementation

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148471724 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10186][SQL] Array types using JDBCRDD a...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9137#issuecomment-148471677 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

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148471682 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42157537 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala --- @@ -305,7 +305,7 @@ private[netty] class NettyRpcEnvFactory extends

[GitHub] spark pull request: [SPARK-11126][SQL]Fix a memory leak in SQLList...

2015-10-15 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9132#issuecomment-148472788 looks good, can you add a test? --- 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 pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148473676 [Test build #43793 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43793/consoleFull) for PR 9126 at commit

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread jodersky
Github user jodersky commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148476176 You were right about NettyRpcEndpointRef, actually conf is not defined as a field in the base class. Sorry I missed this, it is however the case for the listener in

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148476166 hi @dilipbiswal , can you update the PR title to make it a complete sentence(i.e. no ellipsis)? and can you also put my explaination of this bug in PR description? so

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148476797 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10185] [SQL] Feat sql comma separated p...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8416#issuecomment-148476827 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148476915 [Test build #43796 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43796/consoleFull) for PR 9123 at commit

[GitHub] spark pull request: [SPARK-11119][SQL] cleanup for unsafe array an...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9131#issuecomment-148476763 [Test build #43794 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43794/consoleFull) for PR 9131 at commit

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148480193 [Test build #43798 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43798/consoleFull) for PR 9123 at commit

[GitHub] spark pull request: [SPARK-10412][SQL] report memory usage for tun...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8931#issuecomment-148483297 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11131] [core] Fix race in worker regist...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9138#issuecomment-148483296 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10412][SQL] report memory usage for tun...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8931#issuecomment-148484027 [Test build #43801 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43801/consoleFull) for PR 8931 at commit

[GitHub] spark pull request: [SPARK-2365] Add IndexedRDD, an efficient upda...

2015-10-15 Thread tispratik
Github user tispratik commented on the pull request: https://github.com/apache/spark/pull/1297#issuecomment-148489954 This is very interesting. Thanks for working on it. Hopefully it will be out soon. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-10984] [WIP] Simplify *MemoryManager cl...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9127#issuecomment-148490204 [Test build #43789 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43789/console) for PR 9127 at commit

[GitHub] spark pull request: [SPARK-10984] [WIP] Simplify *MemoryManager cl...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9127#issuecomment-148490411 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10984] [WIP] Simplify *MemoryManager cl...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9127#issuecomment-148490409 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

[GitHub] spark pull request: [SPARK-8890][SQL][WIP] Reduce memory consumpti...

2015-10-15 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7514#issuecomment-148490929 Hey @ilganeli @marmbrus, is this PR still relevant after #8010? If not, could we close it? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-8890][SQL][WIP] Reduce memory consumpti...

2015-10-15 Thread ilganeli
Github user ilganeli closed the pull request at: https://github.com/apache/spark/pull/7514 --- 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: [SPARK-11129][MESOS] Link Spark WebUI from Mes...

2015-10-15 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9135#issuecomment-148491984 @tnachen --- 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: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148496260 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

[GitHub] spark pull request: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148496120 [Test build #43791 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43791/console) for PR 8512 at commit

[GitHub] spark pull request: [SPARK-9926] [SPARK-10340] [SQL] Use S3 bulk l...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8512#issuecomment-148496263 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148484211 LGTM pending test. --- 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 pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148484150 [Test build #43798 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43798/console) for PR 9123 at commit

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148484236 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

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148484237 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11131] [core] Fix race in worker regist...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9138#issuecomment-148484709 [Test build #43800 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43800/consoleFull) for PR 9138 at commit

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148486747 hi @marmbrus , checked with @yhuai , `OutputFaker` is no longer used, so I removed it. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148486974 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148487002 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148487000 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148486845 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

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148486981 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148487314 [Test build #43802 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43802/consoleFull) for PR 9123 at commit

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148488344 [Test build #43803 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43803/consoleFull) for PR 9081 at commit

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148496474 [Test build #43803 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43803/console) for PR 9081 at commit

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148496546 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11072][SQL] simplify self join handling

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9081#issuecomment-148496543 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

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148471394 ok to test --- 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: [SPARK-10930] Adds max task duration to all st...

2015-10-15 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9051#issuecomment-148471371 @d2r can we get a screenshot here to help visualize the change? It'd be great to do that for all UI changes. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42157153 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -89,9 +89,9 @@ private[hive] case class CurrentDatabase(ctx:

[GitHub] spark pull request: [SPARK-11119][SQL] cleanup for unsafe array an...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9131#issuecomment-148474034 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11119][SQL] cleanup for unsafe array an...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9131#issuecomment-148474064 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/9123#discussion_r42158784 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala --- @@ -135,4 +135,14 @@ class AnalysisSuite extends

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread jodersky
Github user jodersky commented on a diff in the pull request: https://github.com/apache/spark/pull/9126#discussion_r42158956 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala --- @@ -305,7 +305,7 @@ private[netty] class NettyRpcEnvFactory extends

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148475603 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148475604 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11110] [Build] Remove transient annotat...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9126#issuecomment-148475620 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10534][SQL] ORDER BY clause allows only...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9123#issuecomment-148475647 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

<    1   2   3   4   5   6   7   >