[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132855837
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41296/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132855370
  
  [Test build #41296 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41296/console)
 for   PR 8326 at commit 
[`7a3799b`](https://github.com/apache/spark/commit/7a3799b52737baf2a062a52b76e2552a8fb989c9).
 * This patch **passes all tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9772] [PySpark] [ML] Add Python API for...

2015-08-19 Thread yanboliang
Github user yanboliang commented on the pull request:

https://github.com/apache/spark/pull/8102#issuecomment-132854062
  
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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-19 Thread yanboliang
Github user yanboliang commented on a diff in the pull request:

https://github.com/apache/spark/pull/7976#discussion_r37489806
  
--- Diff: python/pyspark/ml/feature.py ---
@@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel):
 """
 Model fitted by StringIndexer.
 """
+@property
+def labels(self):
+return self._java_obj.labels
+
+
+@inherit_doc
+class IndexToString(JavaTransformer, HasInputCol, HasOutputCol):
+"""
+.. note:: Experimental
+A [[Transformer]] that maps a column of string indices back to a new 
column of corresponding
+string values using either the ML attributes of the input column, or 
if provided using the
+labels supplied by the user.
+All original columns are kept during transformation.
+See L{StringIndexer} for converting strings into indices
+"""
+
+# a placeholder to make the labels show up in generated doc
+labels = Param(Params._dummy(), "labels",
+   "Optional labels to be provided by the user, if not 
supplied column " +
+   "metadata is read for labels. The default value is an 
empty array, " +
+   "but the empty array is ignored and column metadata 
used instead.")
+
+@keyword_only
+def __init__(self, inputCol=None, outputCol=None, labels=[]):
--- End diff --

We should avoid using mutable values ```[]``` as defaults in Python, let's 
use ```None```.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10048][SPARKR] Support arbitrary nested...

2015-08-19 Thread sun-rui
Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/8276#issuecomment-132852075
  
will add test cases.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132851991
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132851994
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41292/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132851909
  
  [Test build #41292 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41292/console)
 for   PR 8324 at commit 
[`8a62b76`](https://github.com/apache/spark/commit/8a62b769c98d4ef22e578c73a01e2311ceca64e8).
 * This patch **passes all tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132850491
  
  [Test build #41297 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41297/consoleFull)
 for   PR 8322 at commit 
[`b801f93`](https://github.com/apache/spark/commit/b801f9345a28e1cb676d6d95f3223201f4d1488d).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848964
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848952
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread tnachen
Github user tnachen commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848583
  
retest this please


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848347
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848348
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41289/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132848260
  
  [Test build #41289 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41289/console)
 for   PR 8322 at commit 
[`b801f93`](https://github.com/apache/spark/commit/b801f9345a28e1cb676d6d95f3223201f4d1488d).
 * This patch **fails PySpark unit tests**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `case class PromotePrecision(child: Expression) extends UnaryExpression 
`
  * `case class CheckOverflow(child: Expression, dataType: DecimalType) 
extends UnaryExpression `



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9689][SQL]Fix bug of not invalidate the...

2015-08-19 Thread chhao01
Github user chhao01 commented on a diff in the pull request:

https://github.com/apache/spark/pull/8023#discussion_r37488201
  
--- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -156,10 +165,27 @@ private[sql] class CacheManager(sqlContext: 
SQLContext) extends Logging {
* function will over invalidate.
*/
   private[sql] def invalidateCache(plan: LogicalPlan): Unit = writeLock {
-cachedData.foreach {
-  case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
-data.cachedRepresentation.recache()
-  case _ =>
+var i = 0
+var locatedIdx = -1
+// find the index of the cached data, according to the specified 
logical plan
+while (i < cachedData.length && locatedIdx < 0) {
+  cachedData(i) match {
+case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
+  locatedIdx = i
+case _ =>
+  }
+  i += 1
+}
+
+if (locatedIdx >= 0) {
+  // if the cached data exists, remove it from the cache data list, as 
we need to
+  // re-generate the spark plan, and we don't want the this to be used 
during the
+  // re-generation
+  val entry = cachedData.remove(locatedIdx) // TODO do we have to use 
ArrayBuffer?
+  // rebuild the cache
+  entry.recache(sqlContext)
+  // add it back to the cache data list
+  cachedData += entry
--- End diff --

And probably we can not simply use the `data.plan.find`, as the cached data 
most likely will add wrapper like `LogicalPlan` etc.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9812][Streaming]Fix Python 3 compatibil...

2015-08-19 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/8315


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9689][SQL]Fix bug of not invalidate the...

2015-08-19 Thread chhao01
Github user chhao01 commented on a diff in the pull request:

https://github.com/apache/spark/pull/8023#discussion_r37488173
  
--- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -156,10 +165,27 @@ private[sql] class CacheManager(sqlContext: 
SQLContext) extends Logging {
* function will over invalidate.
*/
   private[sql] def invalidateCache(plan: LogicalPlan): Unit = writeLock {
-cachedData.foreach {
-  case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
-data.cachedRepresentation.recache()
-  case _ =>
+var i = 0
+var locatedIdx = -1
+// find the index of the cached data, according to the specified 
logical plan
+while (i < cachedData.length && locatedIdx < 0) {
+  cachedData(i) match {
+case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
+  locatedIdx = i
+case _ =>
+  }
+  i += 1
+}
+
+if (locatedIdx >= 0) {
+  // if the cached data exists, remove it from the cache data list, as 
we need to
+  // re-generate the spark plan, and we don't want the this to be used 
during the
+  // re-generation
+  val entry = cachedData.remove(locatedIdx) // TODO do we have to use 
ArrayBuffer?
+  // rebuild the cache
+  entry.recache(sqlContext)
+  // add it back to the cache data list
+  cachedData += entry
--- End diff --

yes, that's a good catch, I will see how to fix the chained logical cached 
plan.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132845139
  
  [Test build #41296 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41296/consoleFull)
 for   PR 8326 at commit 
[`7a3799b`](https://github.com/apache/spark/commit/7a3799b52737baf2a062a52b76e2552a8fb989c9).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread tdas
Github user tdas commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132844994
  
LGTM! Will merge when tests pass.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9689][SQL]Fix bug of not invalidate the...

2015-08-19 Thread chhao01
Github user chhao01 commented on a diff in the pull request:

https://github.com/apache/spark/pull/8023#discussion_r37487843
  
--- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelation.scala
 ---
@@ -160,6 +160,9 @@ private[sql] case class InsertIntoHadoopFsRelation(
   logInfo("Skipping insertion into a relation that already exists.")
 }
 
+// Invalidate the cache.
+sqlContext.cacheManager.invalidateCache(LogicalRelation(relation))
--- End diff --

Yes, true, I will update it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10126] [PROJECT INFRA] Fix typo in rele...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8325#issuecomment-132844686
  
  [Test build #41295 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41295/consoleFull)
 for   PR 8325 at commit 
[`cdcb6e5`](https://github.com/apache/spark/commit/cdcb6e52a5e402eca96bc5b39c6839397684f699).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10126] [PROJECT INFRA] Fix typo in rele...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8325#issuecomment-132844079
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10126] [PROJECT INFRA] Fix typo in rele...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8325#issuecomment-132844067
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132844078
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8326#issuecomment-132844061
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9812][Streaming]Fix Python 3 compatibil...

2015-08-19 Thread tdas
Github user tdas commented on the pull request:

https://github.com/apache/spark/pull/8315#issuecomment-132844018
  
Thanks! merging 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10125][Streaming]Fix a potential deadlo...

2015-08-19 Thread zsxwing
GitHub user zsxwing opened a pull request:

https://github.com/apache/spark/pull/8326

[SPARK-10125][Streaming]Fix a potential deadlock in JobGenerator.stop

Because `lazy val` uses `this` lock, if JobGenerator.stop and 
JobGenerator.doCheckpoint (JobGenerator.shouldCheckpoint has not yet been 
initialized) run at the same time, it may hang.

Here are the stack traces for the deadlock:

```Java
"pool-1-thread-1-ScalaTest-running-StreamingListenerSuite" #11 prio=5 
os_prio=31 tid=0x7fd35d094800 nid=0x5703 in Object.wait() 
[0x00012ecaf000]
   java.lang.Thread.State: WAITING (on object monitor)
at java.lang.Object.wait(Native Method)
at java.lang.Thread.join(Thread.java:1245)
- locked <0x0007b5d8d7f8> (a 
org.apache.spark.util.EventLoop$$anon$1)
at java.lang.Thread.join(Thread.java:1319)
at org.apache.spark.util.EventLoop.stop(EventLoop.scala:81)
at 
org.apache.spark.streaming.scheduler.JobGenerator.stop(JobGenerator.scala:155)
- locked <0x0007b5d8cea0> (a 
org.apache.spark.streaming.scheduler.JobGenerator)
at 
org.apache.spark.streaming.scheduler.JobScheduler.stop(JobScheduler.scala:95)
- locked <0x0007b5d8ced8> (a 
org.apache.spark.streaming.scheduler.JobScheduler)
at 
org.apache.spark.streaming.StreamingContext.stop(StreamingContext.scala:687)

"JobGenerator" #67 daemon prio=5 os_prio=31 tid=0x7fd35c3b9800 
nid=0x9f03 waiting for monitor entry [0x000139e4a000]
   java.lang.Thread.State: BLOCKED (on object monitor)
at 
org.apache.spark.streaming.scheduler.JobGenerator.shouldCheckpoint$lzycompute(JobGenerator.scala:63)
- waiting to lock <0x0007b5d8cea0> (a 
org.apache.spark.streaming.scheduler.JobGenerator)
at 
org.apache.spark.streaming.scheduler.JobGenerator.shouldCheckpoint(JobGenerator.scala:63)
at 
org.apache.spark.streaming.scheduler.JobGenerator.doCheckpoint(JobGenerator.scala:290)
at 
org.apache.spark.streaming.scheduler.JobGenerator.org$apache$spark$streaming$scheduler$JobGenerator$$processEvent(JobGenerator.scala:182)
at 
org.apache.spark.streaming.scheduler.JobGenerator$$anon$1.onReceive(JobGenerator.scala:83)
at 
org.apache.spark.streaming.scheduler.JobGenerator$$anon$1.onReceive(JobGenerator.scala:82)
at org.apache.spark.util.EventLoop$$anon$1.run(EventLoop.scala:48)
```

I can use this patch to produce this deadlock: 
https://github.com/zsxwing/spark/commit/8a88f28d1331003a65fabef48ae3d22a7c21f05f

And a timeout build in Jenkins due to this deadlock: 
https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1654/

This PR initializes `checkpointWriter` before `eventLoop` uses it to avoid 
this deadlock.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zsxwing/spark SPARK-10125

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/8326.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #8326


commit 7a3799b52737baf2a062a52b76e2552a8fb989c9
Author: zsxwing 
Date:   2015-08-20T01:18:34Z

Fix a potential deadlock in JobGenerator.stop




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10053] Export lapply and lapplyPartitio...

2015-08-19 Thread sun-rui
Github user sun-rui commented on the pull request:

https://github.com/apache/spark/pull/8240#issuecomment-132843607
  
I think UDF() is important, hope it can be available in 1.6 release.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10126] [PROJECT INFRA] Fix typo in rele...

2015-08-19 Thread JoshRosen
GitHub user JoshRosen opened a pull request:

https://github.com/apache/spark/pull/8325

[SPARK-10126] [PROJECT INFRA] Fix typo in release-build.sh which broke 
snapshot publishing for Scala 2.11

The current `release-build.sh` has a typo which breaks snapshot publication 
for Scala 2.11. We should change the Scala version to 2.11 and clean before 
building a 2.11 snapshot.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/JoshRosen/spark fix-2.11-snapshots

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/8325.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #8325


commit cdcb6e52a5e402eca96bc5b39c6839397684f699
Author: Josh Rosen 
Date:   2015-08-20T01:18:11Z

Fix typos release-build.sh




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9689][SQL]Fix bug of not invalidate the...

2015-08-19 Thread chhao01
Github user chhao01 commented on a diff in the pull request:

https://github.com/apache/spark/pull/8023#discussion_r37487474
  
--- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -156,10 +165,27 @@ private[sql] class CacheManager(sqlContext: 
SQLContext) extends Logging {
* function will over invalidate.
*/
   private[sql] def invalidateCache(plan: LogicalPlan): Unit = writeLock {
-cachedData.foreach {
-  case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
-data.cachedRepresentation.recache()
-  case _ =>
+var i = 0
+var locatedIdx = -1
+// find the index of the cached data, according to the specified 
logical plan
+while (i < cachedData.length && locatedIdx < 0) {
+  cachedData(i) match {
+case data if data.plan.collect { case p if p.sameResult(plan) => p 
}.nonEmpty =>
+  locatedIdx = i
+case _ =>
+  }
+  i += 1
+}
+
+if (locatedIdx >= 0) {
+  // if the cached data exists, remove it from the cache data list, as 
we need to
+  // re-generate the spark plan, and we don't want the this to be used 
during the
+  // re-generation
--- End diff --

In `SQLContext.QueryExecution`, we always try to load the cached 
`PhysicalRDD` according to the logical plan first, if we don't remove it from 
the cached list, then we will always get the stale one, which is not what we 
want, right?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9400][SQL]Codegen for StringLocate

2015-08-19 Thread yjshen
Github user yjshen commented on a diff in the pull request:

https://github.com/apache/spark/pull/8274#discussion_r37487469
  
--- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
 ---
@@ -582,6 +582,109 @@ case class StringLocate(substr: Expression, str: 
Expression, start: Expression)
 }
   }
 
+  override def genCode(ctx: CodeGenContext, ev: GeneratedExpressionCode): 
String = {
+if (start.foldable) {
--- End diff --

I was following 
https://github.com/apache/spark/pull/7779#discussion_r35945742, if it's not 
required, I think 7779 is fine.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9689][SQL]Fix bug of not invalidate the...

2015-08-19 Thread chhao01
Github user chhao01 commented on a diff in the pull request:

https://github.com/apache/spark/pull/8023#discussion_r37487286
  
--- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -97,13 +106,13 @@ private[sql] class CacheManager(sqlContext: 
SQLContext) extends Logging {
   logWarning("Asked to cache already cached data.")
 } else {
   cachedData +=
-CachedData(
+new CachedData(
   planToCache,
   InMemoryRelation(
 sqlContext.conf.useCompression,
 sqlContext.conf.columnBatchSize,
 storageLevel,
-sqlContext.executePlan(query.logicalPlan).executedPlan,
+sqlContext.executePlan(planToCache).executedPlan,
--- End diff --

This is just an optimization, and we don't want to re-analysis the logical 
plan as it's done right before the calls.

When you check the full code of the function `cacheQuery`, you will see the 
`planToCache` is just a local variable, not a logical plan cached by somewhere 
else, it should not be the case you were talking.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132838921
  
  [Test build #41293 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41293/consoleFull)
 for   PR 8320 at commit 
[`200afdc`](https://github.com/apache/spark/commit/200afdc33f59508a04172cfefcc4a40a3bdf6169).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9400][SQL]Codegen for StringLocate

2015-08-19 Thread davies
Github user davies commented on a diff in the pull request:

https://github.com/apache/spark/pull/8274#discussion_r37485964
  
--- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
 ---
@@ -582,6 +582,109 @@ case class StringLocate(substr: Expression, str: 
Expression, start: Expression)
 }
   }
 
+  override def genCode(ctx: CodeGenContext, ev: GeneratedExpressionCode): 
String = {
+if (start.foldable) {
--- End diff --

I think we do not need these special cases for StringLocate, because you do 
not need any prepare to call UTF8String.indexOf(substr, start)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132838314
  
  [Test build #41294 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41294/consoleFull)
 for   PR 8239 at commit 
[`fab7042`](https://github.com/apache/spark/commit/fab7042f8b9a11cc9bf3b9b761292c2a92abaa7e).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132838159
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132838183
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132838190
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132838171
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8180#issuecomment-132837778
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8180#issuecomment-132837779
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41286/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8180#issuecomment-132837674
  
  [Test build #41286 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41286/console)
 for   PR 8180 at commit 
[`210ca43`](https://github.com/apache/spark/commit/210ca431ec5e0d51c0e206a5eca549851c470e5e).
 * This patch **passes all tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread shivaram
Github user shivaram commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132837650
  
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9242][SQL] Audit UDAF interface.

2015-08-19 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/8321


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-1267][PYSPARK] Adds pip installer for p...

2015-08-19 Thread holdenk
Github user holdenk commented on a diff in the pull request:

https://github.com/apache/spark/pull/8318#discussion_r37485263
  
--- Diff: python/setup.py ---
@@ -0,0 +1,19 @@
+#!/usr/bin/env python
+
+from setuptools import setup
+
+exec(compile(open("pyspark/pyspark_version.py").read(), 
+   "pyspark/pyspark_version.py", 'exec'))
+VERSION = __version__
+
+setup(name='pyspark',
+version=VERSION,
+description='Apache Spark Python API',
+author='Spark Developers',
+author_email='d...@spark.apache.org',
+url='https://github.com/apache/spark/tree/master/python',
+packages=['pyspark', 'pyspark.mllib', 'pyspark.ml', 'pyspark.sql', 
'pyspark.streaming'],
+data_files=[('pyspark', ['pyspark/pyspark_version.py'])],
+install_requires=['numpy>=1.7', 'py4j==0.8.2.1', 'pandas'],
+license='http://www.apache.org/licenses/LICENSE-2.0',
+)
--- End diff --

This is maybe asking for too much, but in Sparkling Pandas we install our 
own assembly jar*, would it maybe make sense to do that as part of this process?

(*and getting it working has been painful, but doable).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132836236
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132836238
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41282/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9982] [SPARKR] SparkR DataFrame fail to...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8239#issuecomment-132836197
  
**[Test build #41282 timed 
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41282/console)**
 for PR 8239 at commit 
[`fab7042`](https://github.com/apache/spark/commit/fab7042f8b9a11cc9bf3b9b761292c2a92abaa7e)
 after a configured wait of `175m`.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132836139
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41283/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132836087
  
  [Test build #41283 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41283/console)
 for   PR 8320 at commit 
[`06c8ffa`](https://github.com/apache/spark/commit/06c8ffa6f576baad66b5897756eda990f114ade4).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `public class LargeByteBufferInputStream extends InputStream `
  * `public class LargeByteBufferOutputStream extends OutputStream `
  * `abstract class BlockSizeLimitException(msg: String, cause: 
BufferTooLargeException)`
  * `class ReplicationBlockSizeLimitException(cause: 
BufferTooLargeException)`
  * `class TachyonBlockSizeLimitException(cause: BufferTooLargeException)`
  * `class ShuffleBlockSizeLimitException(size: Long)`
  * `class ShuffleRemoteBlockSizeLimitException(cause: 
BufferTooLargeException)`
  * `public class BufferTooLargeException extends IOException `
  * `public class LargeByteBufferHelper `
  * `public class WrappedLargeByteBuffer implements LargeByteBuffer `



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6236] [core] [wip] caching for blocks o...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8320#issuecomment-132836138
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10017] [MLlib]: ML model broadcasts sho...

2015-08-19 Thread holdenk
Github user holdenk commented on the pull request:

https://github.com/apache/spark/pull/8241#issuecomment-132836131
  
Glad to help, maybe @jkbradley can say if this is ok to test to jenkins :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9242][SQL] Audit UDAF interface.

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8321#issuecomment-132834417
  
  [Test build #1673 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1673/console)
 for   PR 8321 at commit 
[`8d29a42`](https://github.com/apache/spark/commit/8d29a42926ec6452e0f37cbbf0c4d7b106e2ce5b).
 * This patch **passes all tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10035][SQL] Parquet filters does not pr...

2015-08-19 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/8275


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10035][SQL] Parquet filters does not pr...

2015-08-19 Thread liancheng
Github user liancheng commented on the pull request:

https://github.com/apache/spark/pull/8275#issuecomment-132830954
  
Thanks! Merging to master (not to branch-1.5 at this moment since this is a 
new feature).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132830847
  
  [Test build #41292 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41292/consoleFull)
 for   PR 8324 at commit 
[`8a62b76`](https://github.com/apache/spark/commit/8a62b769c98d4ef22e578c73a01e2311ceca64e8).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132830468
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread yhuai
GitHub user yhuai opened a pull request:

https://github.com/apache/spark/pull/8324

[SPARK-10092] [SQL] Multi-DB support follow up.

https://issues.apache.org/jira/browse/SPARK-10092

This pr is a follow-up one for Multi-DB support. It has the following 
changes:

* `Hive.Context.refreshTable` now accepts `dbName.tableName`.
* `Hive.Context.analyze` now accepts `dbName.tableName`.
* `CreateTableUsing`, `CreateTableUsingAsSelect`, `CreateTempTableUsing`, 
`CreateTempTableUsingAsSelect`, `CreateMetastoreDataSource`, and 
`CreateMetastoreDataSourceAsSelect` all take `TableIdentifier` instead of the 
string representation of table name.
* When you call `saveAsTable` with a specified database, the data will be 
saved to the correct location.
* Explicitly do not allow user to create a temporary with a specified 
database name (users cannot do it before).

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/yhuai/spark saveAsTableDB

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/8324.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #8324


commit 8a62b769c98d4ef22e578c73a01e2311ceca64e8
Author: Yin Huai 
Date:   2015-08-19T23:54:24Z

Multi-db support follow up.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10092] [SQL] Multi-DB support follow up...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8324#issuecomment-132830450
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132828228
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41291/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132828227
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132828223
  
  [Test build #41291 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41291/console)
 for   PR 8323 at commit 
[`d4f48ba`](https://github.com/apache/spark/commit/d4f48ba4855fbb6f21c50924475b3810df048826).
 * This patch **fails RAT tests**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `class _DelegateIO(object):`
  * `class _TestInfo(object):`
  * `class _XMLTestResult(_TextTestResult):`
  * `class XMLTestRunner(TextTestRunner):`



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132828208
  
  [Test build #41291 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41291/consoleFull)
 for   PR 8323 at commit 
[`d4f48ba`](https://github.com/apache/spark/commit/d4f48ba4855fbb6f21c50924475b3810df048826).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread gliptak
Github user gliptak commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132827862
  
I just added the headers, xmlrunner is LGPL (and only used to run the 
tests).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread JoshRosen
Github user JoshRosen commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132827574
  
The build failed on RAT:

```
Could not find Apache license headers in the following files:
 !? 
/home/jenkins/workspace/SparkPullRequestBuilder/dev/xmlrunner/__init__.py
 !? 
/home/jenkins/workspace/SparkPullRequestBuilder/dev/xmlrunner/xmlrunner.py
[error] running 
/home/jenkins/workspace/SparkPullRequestBuilder/dev/check-license ; received 
return code 1
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132827411
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132827392
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread gliptak
Github user gliptak commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132825406
  

https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/console
Recording test results
ERROR: Publisher 'Publish JUnit test result report' failed: No test report 
files were found. Configuration error?
Test FAILed.

Can somebody comment on how is Jenkins configured to look for JUnit results 
(or did this build fail on RAT)? 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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132823658
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132823659
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41277/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132823574
  
**[Test build #41277 timed 
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41277/console)**
 for PR 7739 at commit 
[`bca4fa3`](https://github.com/apache/spark/commit/bca4fa303a6de1b8b395c9fc6c2357edb9e12d5e)
 after a configured wait of `175m`.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132821458
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132821459
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41275/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132821390
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132821352
  
  [Test build #41290 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/consoleFull)
 for   PR 8323 at commit 
[`d93e7cd`](https://github.com/apache/spark/commit/d93e7cdefda962191a3e615708f977750799422e).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132821391
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/7739#issuecomment-132821326
  
  [Test build #41275 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41275/console)
 for   PR 7739 at commit 
[`5f62d39`](https://github.com/apache/spark/commit/5f62d39aadafc1780570c368fe6ceeaa39186758).
 * This patch **passes all tests**.
 * This patch **does not merge cleanly**.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132821383
  
  [Test build #41290 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41290/console)
 for   PR 8323 at commit 
[`d93e7cd`](https://github.com/apache/spark/commit/d93e7cdefda962191a3e615708f977750799422e).
 * This patch **fails RAT tests**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `class _DelegateIO(object):`
  * `class _TestInfo(object):`
  * `class _XMLTestResult(_TextTestResult):`
  * `class XMLTestRunner(TextTestRunner):`



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6190][core] create LargeByteBuffer for ...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/5400#issuecomment-132820924
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41274/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6190][core] create LargeByteBuffer for ...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/5400#issuecomment-132820920
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-6190][core] create LargeByteBuffer for ...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/5400#issuecomment-132820852
  
**[Test build #41274 timed 
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41274/console)**
 for PR 5400 at commit 
[`80c4032`](https://github.com/apache/spark/commit/80c40324e266590a03971b0e8002d8c9ac585eea)
 after a configured wait of `175m`.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132819515
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10035][SQL] Parquet filters does not pr...

2015-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request:

https://github.com/apache/spark/pull/8275#issuecomment-132819486
  
Could you 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132819556
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread JoshRosen
Github user JoshRosen commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132819078
  
Jenkins, this is 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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9846] [Docs] User guide for Multilayer ...

2015-08-19 Thread avulanov
Github user avulanov commented on the pull request:

https://github.com/apache/spark/pull/8262#issuecomment-132818915
  
The test fail seems unrelated to my commit.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9846] [Docs] User guide for Multilayer ...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8262#issuecomment-132817697
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9846] [Docs] User guide for Multilayer ...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8262#issuecomment-132817708
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41288/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132816388
  
  [Test build #41289 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41289/consoleFull)
 for   PR 8322 at commit 
[`b801f93`](https://github.com/apache/spark/commit/b801f9345a28e1cb676d6d95f3223201f4d1488d).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread gliptak
GitHub user gliptak opened a pull request:

https://github.com/apache/spark/pull/8323

SPARK-7021 Add JUnit output for Python tests

WIP

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gliptak/spark SPARK-7021

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/8323.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #8323


commit d93e7cdefda962191a3e615708f977750799422e
Author: Gábor Lipták 
Date:   2015-08-18T01:20:19Z

SPARK-7021 Add JUnit output for Python tests




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: SPARK-7021 Add JUnit output for Python tests

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8323#issuecomment-132816243
  
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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread andrewor14
Github user andrewor14 commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132816122
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132815499
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132815481
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread tnachen
Github user tnachen commented on the pull request:

https://github.com/apache/spark/pull/8322#issuecomment-132815064
  
@andrewor14 PTAL!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-10124] [MESOS] Fix removing queued driv...

2015-08-19 Thread tnachen
GitHub user tnachen opened a pull request:

https://github.com/apache/spark/pull/8322

[SPARK-10124] [MESOS] Fix removing queued driver in mesos cluster mode.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tnachen/spark fix_cluster_mode

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/8322.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #8322


commit b801f9345a28e1cb676d6d95f3223201f4d1488d
Author: Timothy Chen 
Date:   2015-08-19T22:42:47Z

Fix removing queued driver in mesos cluster mode.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9846] [Docs] User guide for Multilayer ...

2015-08-19 Thread avulanov
Github user avulanov commented on the pull request:

https://github.com/apache/spark/pull/8262#issuecomment-132814397
  
@mengxr Thank you for your comments! I've addressed everything except the 
full chop of setters for the Java trainer. It complains on `java: incompatible 
types: org.apache.spark.ml.classification.MultilayerPerceptronParams cannot be 
converted to 
org.apache.spark.ml.classification.MultilayerPerceptronClassifier`. I did it in 
two stages. Also I've left Java imports.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9895] User Guide for RFormula Feature T...

2015-08-19 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/8293


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-9895] User Guide for RFormula Feature T...

2015-08-19 Thread mengxr
Github user mengxr commented on the pull request:

https://github.com/apache/spark/pull/8293#issuecomment-132813989
  
LGTM. Merged 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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



<    1   2   3   4   5   6   7   8   >