[GitHub] spark issue #14275: [SPARK-16637] Unified containerizer

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

[GitHub] spark pull request #14275: [SPARK-16637] Unified containerizer

2016-07-25 Thread mgummelt
Github user mgummelt commented on a diff in the pull request: https://github.com/apache/spark/pull/14275#discussion_r72141303 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtil.scala --- @@ -105,16 +105,27 @@ private[mesos] object

[GitHub] spark issue #14350: [SPARK-16715][Tests]Fix a potential ExprId conflict for ...

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

[GitHub] spark issue #14349: [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGener...

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

[GitHub] spark pull request #14350: [SPARK-16715][Tests]Fix a potential ExprId confli...

2016-07-25 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/14350 [SPARK-16715][Tests]Fix a potential ExprId conflict for SubexpressionEliminationSuite."Semantic equals and hash" ## What changes were proposed in this pull request?

[GitHub] spark issue #14349: [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGener...

2016-07-25 Thread ooq
Github user ooq commented on the issue: https://github.com/apache/spark/pull/14349 This PR is a cleaner version for 14174: https://github.com/apache/spark/pull/14174. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #14174: [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGener...

2016-07-25 Thread ooq
Github user ooq commented on the issue: https://github.com/apache/spark/pull/14174 Closing this now. Open another PR for cleaner commit history: https://github.com/apache/spark/pull/14349 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request #14174: [SPARK-16524][SQL] Add RowBatch and RowBasedHashM...

2016-07-25 Thread ooq
Github user ooq closed the pull request at: https://github.com/apache/spark/pull/14174 --- 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 #14132: [SPARK-16475][SQL] Broadcast Hint for SQL Queries

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

[GitHub] spark issue #14132: [SPARK-16475][SQL] Broadcast Hint for SQL Queries

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14132 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 pull request #14349: [SPARK-16524][SQL] Add RowBatch and RowBasedHashM...

2016-07-25 Thread ooq
GitHub user ooq opened a pull request: https://github.com/apache/spark/pull/14349 [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGenerator ## What changes were proposed in this pull request? This PR is the first step for the following feature: For hash

[GitHub] spark issue #14132: [SPARK-16475][SQL] Broadcast Hint for SQL Queries

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

[GitHub] spark pull request #14298: [SPARK-16283][SQL] Implement `percentile_approx` ...

2016-07-25 Thread thunterdb
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/14298#discussion_r72137984 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/PercentileApprox.scala --- @@ -0,0 +1,456 @@ +/* + *

[GitHub] spark pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread jaceklaskowski
Github user jaceklaskowski commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72137376 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala --- @@ -269,19 +273,11 @@ class StreamExecution(

[GitHub] spark issue #14348: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14348 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

[GitHub] spark issue #14309: [SPARK-11977][SQL] Support accessing a column contains "...

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

[GitHub] spark issue #14309: [SPARK-11977][SQL] Support accessing a column contains "...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14309 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 #14309: [SPARK-11977][SQL] Support accessing a column contains "...

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

[GitHub] spark pull request #14348: [SPARK-15271] [MESOS] Allow force pulling executo...

2016-07-25 Thread philipphoffmann
GitHub user philipphoffmann opened a pull request: https://github.com/apache/spark/pull/14348 [SPARK-15271] [MESOS] Allow force pulling executor docker images ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ##

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-07-25 Thread philipphoffmann
Github user philipphoffmann commented on the issue: https://github.com/apache/spark/pull/13051 fixed the above issue and sent another request here: #14348 --- 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 #13920: [SPARK-16166][Core] Also take off-heap memory usage into...

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

[GitHub] spark issue #13920: [SPARK-16166][Core] Also take off-heap memory usage into...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/13920 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 issue #14174: [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGener...

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

[GitHub] spark pull request #14270: [SPARK-5847][CORE] Allow for configuring MetricsS...

2016-07-25 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/14270#discussion_r72131840 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -103,4 +103,9 @@ package object config { .stringConf

[GitHub] spark pull request #14297: [SPARK-16660][SQL] CreateViewCommand should not t...

2016-07-25 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14297#discussion_r72131816 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -44,7 +50,11 @@ import

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/13051 Shoot, yeah PR builder tests passed, but that was 7 days ago. Sorry @philipphoffmann can you try this one again after figuring out whatever caused this? --- If your project is set up for it, you

[GitHub] spark pull request #14345: [CORE] Added new high-order function to RDD: cart...

2016-07-25 Thread piffall
Github user piffall closed the pull request at: https://github.com/apache/spark/pull/14345 --- 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 #14345: [CORE] Added new high-order function to RDD: cartesianFi...

2016-07-25 Thread piffall
Github user piffall commented on the issue: https://github.com/apache/spark/pull/14345 Ok. 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 enabled and wishes so, or if

[GitHub] spark issue #14257: [SPARK-16621][SQL] Generate stable SQLs in SQLBuilder

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14257 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 #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/13051 This patch just broke the build: ``` [error]

[GitHub] spark issue #14257: [SPARK-16621][SQL] Generate stable SQLs in SQLBuilder

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

[GitHub] spark issue #14257: [SPARK-16621][SQL] Generate stable SQLs in SQLBuilder

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

[GitHub] spark issue #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avoiding p...

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

[GitHub] spark issue #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avoiding p...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14292 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 #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avoiding p...

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

[GitHub] spark issue #13571: [SPARK-15369][WIP][RFC][PySpark][SQL] Expose potential t...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13571 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 #13571: [SPARK-15369][WIP][RFC][PySpark][SQL] Expose potential t...

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

[GitHub] spark issue #13571: [SPARK-15369][WIP][RFC][PySpark][SQL] Expose potential t...

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

[GitHub] spark issue #13886: [SPARK-16185] [SQL] Better Error Messages When Creating ...

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

[GitHub] spark issue #13886: [SPARK-16185] [SQL] Better Error Messages When Creating ...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13886 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 #13886: [SPARK-16185] [SQL] Better Error Messages When Creating ...

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

[GitHub] spark pull request #14194: [SPARK-16485][DOC][ML] Fixed several inline forma...

2016-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14194 --- 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 #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avoiding p...

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

[GitHub] spark issue #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14194 Merged to master/2.0 to match original change --- 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 #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

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

[GitHub] spark issue #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14194 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 #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

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

[GitHub] spark pull request #14309: [SPARK-11977][SQL] Support accessing a column con...

2016-07-25 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/14309#discussion_r72127680 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -896,6 +896,19 @@ class DataFrameSuite extends QueryTest with

[GitHub] spark issue #13571: [SPARK-15369][WIP][RFC][PySpark][SQL] Expose potential t...

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

[GitHub] spark pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72127350 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/HDFSMetadataLog.scala --- @@ -91,18 +92,30 @@ class HDFSMetadataLog[T:

[GitHub] spark pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72127293 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala --- @@ -269,19 +273,11 @@ class StreamExecution( *

[GitHub] spark issue #14333: [SPARK-16696][ML][MLLib] destroy KMeans bcNewCenters whe...

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14333 OK sounds good. So maybe we're back to this: for `bcNewCenters`, is it really worth the overhead to track and destroy them? or just settle for unpersisting within the loop? I could go either way,

[GitHub] spark issue #14275: [SPARK-16637] Unified containerizer

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14275 Looks like one more last comment, and needs a rebase after https://github.com/apache/spark/pull/13051 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #14307: [SPARK-16672][SQL] SQLBuilder should not raise exception...

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

[GitHub] spark issue #14307: [SPARK-16672][SQL] SQLBuilder should not raise exception...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14307 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 pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72126474 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/test/SQLTestUtils.scala --- @@ -247,6 +248,46 @@ private[sql] trait SQLTestUtils } }

[GitHub] spark pull request #13051: [SPARK-15271] [MESOS] Allow force pulling executo...

2016-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13051 --- 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 #14307: [SPARK-16672][SQL] SQLBuilder should not raise exception...

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

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

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

[GitHub] spark issue #14345: [CORE] Added new high-order function to RDD: cartesianFi...

2016-07-25 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14345 So I'm tempted to say no to this API addition because once we want to do things like this, we should really rely on an optimizer to push things automatically, which is the point for SQL and the

[GitHub] spark issue #14257: [SPARK-16621][SQL] Generate stable SQLs in SQLBuilder

2016-07-25 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14257 cc @liancheng --- 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

[GitHub] spark pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72125645 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/HDFSMetadataLog.scala --- @@ -91,18 +92,30 @@ class HDFSMetadataLog[T:

[GitHub] spark pull request #14292: [SPARK-14131][SQL[STREAMING] Improved fix for avo...

2016-07-25 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/14292#discussion_r72125535 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/HDFSMetadataLog.scala --- @@ -91,18 +92,30 @@ class HDFSMetadataLog[T:

[GitHub] spark issue #13758: [SPARK-16043][SQL] Prepare GenericArrayData implementati...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13758 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 #13758: [SPARK-16043][SQL] Prepare GenericArrayData implementati...

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

[GitHub] spark issue #13758: [SPARK-16043][SQL] Prepare GenericArrayData implementati...

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

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72123421 --- Diff: python/pyspark/streaming/kafka010.py --- @@ -0,0 +1,370 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] spark pull request #14342: [SPARK-16685] Remove audit-release scripts.

2016-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14342 --- 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 #14340: [SPARK-16534][Streaming][Kafka] Add Python API support f...

2016-07-25 Thread koeninger
Github user koeninger commented on the issue: https://github.com/apache/spark/pull/14340 In terms of feature parity, are we OK with telling python users they are out of luck if they need something not covered by Assign / Subscribe / SubscribePattern? For scala / java users they can

[GitHub] spark issue #14342: [SPARK-16685] Remove audit-release scripts.

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

[GitHub] spark issue #14330: [SPARK-16693][SPARKR] Remove methods deprecated in 2.0.0...

2016-07-25 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/14330 Yeah lets discuss this on JIRA / mailing lists and then get back to 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

[GitHub] spark pull request #14286: [SPARK-16653][ML][Optimizer] update ANN convergen...

2016-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14286 --- 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 #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

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

[GitHub] spark issue #14194: [SPARK-16485][DOC][ML] Fixed several inline formatting i...

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14194 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 wishes

[GitHub] spark issue #14286: [SPARK-16653][ML][Optimizer] update ANN convergence tole...

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

[GitHub] spark issue #13824: [SPARK-16110][YARN][PYSPARK] Fix allowing python version...

2016-07-25 Thread KevinGrealish
Github user KevinGrealish commented on the issue: https://github.com/apache/spark/pull/13824 @vanzin the function setupLaunchEnv reads env vars directly using sys.env.get so I don't understand how SparkConf helps you setup values for this function to read. See my earlier link to

[GitHub] spark issue #13920: [SPARK-16166][Core] Also take off-heap memory usage into...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/13920 Sorry for the delay in reviewing; I've been super busy and stuff has slipped through the cracks. LGTM because this looks like a net improvement over what we have today. In the longer term I think

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72122440 --- Diff: python/pyspark/streaming/kafka010.py --- @@ -0,0 +1,370 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72122274 --- Diff: python/pyspark/streaming/kafka010.py --- @@ -0,0 +1,370 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] spark issue #13824: [SPARK-16110][YARN][PYSPARK] Fix allowing python version...

2016-07-25 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/13824 You could change that code to use `sparkConf.getenv`, since there's a SparkConf instance available. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request #13620: [SPARK-15590] [WEBUI] Paginate Job Table in Jobs ...

2016-07-25 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/13620#discussion_r72121948 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala --- @@ -210,64 +214,69 @@ private[ui] class AllJobsPage(parent: JobsTab) extends

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72121957 --- Diff: python/pyspark/streaming/kafka010.py --- @@ -0,0 +1,370 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] spark pull request #14204: [SPARK-16520] [WEBUI] Link executors to correspon...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/14204#discussion_r72121585 --- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala --- @@ -934,7 +935,8 @@ private[spark] object JsonProtocol { val

[GitHub] spark pull request #14204: [SPARK-16520] [WEBUI] Link executors to correspon...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/14204#discussion_r72121393 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/ExecutorData.scala --- @@ -34,5 +34,6 @@ private[cluster] class ExecutorData(

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72120535 --- Diff: external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/KafkaUtils.scala --- @@ -177,3 +182,172 @@ object KafkaUtils extends

[GitHub] spark issue #14347: [SPARK-16712] Update Ivy artifacts statement for dev/aud...

2016-07-25 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14347 @jayunit100 this whole directory is about to be deleted anyway, so I think this is redundant. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #12374: [SPARK-14610][ML] Remove superfluous split for continuou...

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

[GitHub] spark issue #12374: [SPARK-14610][ML] Remove superfluous split for continuou...

2016-07-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/12374 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 #12374: [SPARK-14610][ML] Remove superfluous split for continuou...

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

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72118143 --- Diff: external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/KafkaUtils.scala --- @@ -177,3 +182,172 @@ object KafkaUtils extends

[GitHub] spark issue #14311: [SPARK-16550] [core] Certain classes fail to deserialize...

2016-07-25 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/14311 @ericl, these test failures appear legitimate. --- 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 #14329: [SPARKR][DOCS] fix broken url in doc

2016-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14329 --- 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 #14132: [SPARK-16475][SQL] Broadcast Hint for SQL Queries

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

[GitHub] spark issue #14329: [SPARKR][DOCS] fix broken url in doc

2016-07-25 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/14329 Thanks @felixcheung - LGTM. Merging this to master, 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

[GitHub] spark issue #14309: [SPARK-11977][SQL] Support accessing a column contains "...

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

[GitHub] spark pull request #13704: [SPARK-15985][SQL] Eliminate redundant cast from ...

2016-07-25 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/13704#discussion_r72116259 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1441,6 +1441,12 @@ object PushPredicateThroughJoin

[GitHub] spark issue #14345: [CORE] Added new high-order function to RDD: cartesianFi...

2016-07-25 Thread piffall
Github user piffall commented on the issue: https://github.com/apache/spark/pull/14345 ```scala val rdd1 = sc.parallelize(1 to 10) val rdd2 = sc.parallelize(1 to 10) val countCartesian = rdd1.cartesian(rdd2).filter(x => x._1 == x._2).count() val

[GitHub] spark pull request #14340: [SPARK-16534][Streaming][Kafka] Add Python API su...

2016-07-25 Thread koeninger
Github user koeninger commented on a diff in the pull request: https://github.com/apache/spark/pull/14340#discussion_r72115899 --- Diff: examples/src/main/python/streaming/direct_kafka010_wordcount.py --- @@ -0,0 +1,59 @@ +# +# Licensed to the Apache Software Foundation

[GitHub] spark issue #14309: [SPARK-11977][SQL] Support accessing a column contains "...

2016-07-25 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/14309 Jenkins, 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 enabled and wishes

[GitHub] spark pull request #14132: [SPARK-16475][SQL] Broadcast Hint for SQL Queries

2016-07-25 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/14132#discussion_r72114534 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1774,6 +1775,49 @@ class Analyzer( }

<    1   2   3   4   5   6   7   >