Github user ash211 commented on the pull request:
https://github.com/apache/spark/pull/1281#issuecomment-48102838
Hi @YanTangZhai, with the merge of
https://github.com/apache/spark/pull/1274 is this change still needed?
---
If your project is set up for it, you can reply to this emai
Github user YanjieGao commented on the pull request:
https://github.com/apache/spark/pull/1306#issuecomment-48100374
Thanks ,I optimize the code so it only evaluates the function once .Other
comments are on JIRA
---
If your project is set up for it, you can reply to this email and ha
Github user YanjieGao commented on the pull request:
https://github.com/apache/spark/pull/1150#issuecomment-48099853
Thanks a lot, I have reformat the code style.
---
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 pr
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1238#discussion_r14573147
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
---
@@ -26,6 +26,17 @@ import org.apache.spark.sq
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1238#discussion_r14573140
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -61,8 +61,6 @@ private[sql] abstract class SparkStrategi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48098033
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16356/
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48098032
Merged build finished. All automated tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48097436
Merged build finished. All automated tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48097437
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16355/
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48096080
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48096085
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
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1308#discussion_r14572630
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/json/JsonSuite.scala
---
@@ -448,9 +448,9 @@ class JsonSuite extends QueryTest {
}
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1150#issuecomment-48095551
> Thanks ,I add three blank after that line ,Now there aren't red lines.
If there is trailing whitespace we should remove it. It is okay if it is
not code you a
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572610
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -385,6 +385,17 @@ class SQLQuerySuite extends QueryTest {
sql("S
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572609
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -385,6 +385,17 @@ class SQLQuerySuite extends QueryTest {
sql("S
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572607
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -220,3 +220,16 @@ case class Except(left: SparkPlan, right: Spar
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572602
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/SqlParser.scala ---
@@ -140,6 +141,7 @@ class SqlParser extends StandardTokenParsers with
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572606
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -282,6 +282,8 @@ private[sql] abstract class SparkStrategies ex
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1150#discussion_r14572604
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -282,6 +282,8 @@ private[sql] abstract class SparkStrategies ex
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48095474
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1308#issuecomment-48095478
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
GitHub user yhuai opened a pull request:
https://github.com/apache/spark/pull/1308
[SPARK-2375][SQL] JSON schema inference may not resolve type conflicts
correctly for a field inside an array of structs
For example, for
```
{"array": [{"field":214748364700}, {"field":1}]}
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1266#issuecomment-48093749
Thanks! I've merge this into master and we'll back port it into 1.0 after
the 1.0.1 release.
---
If your project is set up for it, you can reply to this email and have
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1266
---
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 enab
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1301#issuecomment-48093617
Thanks! Merged into master and 1.0.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1301
---
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 enab
Github user karlhigley commented on the pull request:
https://github.com/apache/spark/pull/304#issuecomment-48093398
This looked useful, so I tried it out. It works as expected so long as the
feature array is an `Array[Any]`.If the features are all categorical, then
the feature a
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1276#issuecomment-48090143
The cause seems to be that when you do operations like map() followed by
map(), you get a PipelinedRDD, which does not necessarily have an underlying
Java RDD until you ac
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083786
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16354/
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083785
Merged build finished. All automated tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083120
Merged build finished.
---
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083121
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16353/
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083048
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48083043
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48082949
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1307#issuecomment-48082948
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 h
GitHub user ankurdave opened a pull request:
https://github.com/apache/spark/pull/1307
[SPARK-2374] Path pattern matching for GraphX
Based on a
[request](http://apache-spark-user-list.1001560.n3.nabble.com/Graphx-traversal-and-merge-interesting-edges-td8788.html)
on the mailing lis
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1306#issuecomment-48080698
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 pro
GitHub user YanjieGao opened a pull request:
https://github.com/apache/spark/pull/1306
[SPARK-2373]RDD add span function (split an RDD to two RDD based on user's
function)]
def span(p: T => Boolean): (RDD[T], RDD[T])
Splits this RDD into a prefix/suffix pair according to a pred
39 matches
Mail list logo