Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10440
---
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 ena
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10441#discussion_r48511839
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -1232,24 +1243,16 @@ private[ui] class TaskPagedTable(
override d
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10441#issuecomment-167675703
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10441#issuecomment-167675690
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10441#issuecomment-167675687
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 projec
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167675496
Thanks. Merging to master and 1.6
---
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 n
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167675193
sql changes look good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10479
---
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 ena
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10368#issuecomment-167673871
As @zsxwing pointed out, the existing unit tests are not sufficient because
they didn't catch the fact that `ShuffleReadMetrics` et al must also be
registered. This
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167673766
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167673765
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167673716
**[Test build #48376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48376/consoleFull)**
for PR 10479 at commit
[`422ef49`](https://g
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10368#discussion_r48511217
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -109,6 +111,9 @@ class KryoSerializer(conf: SparkConf)
kryo
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10441#issuecomment-167672868
@zsxwing, I've pushed a new commit which aims to preserve the old behavior
when increasing the number of items displayed per page while pageNumber > 1;
see fd2d1f2a49
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167672081
Maybe @yhuai or @liancheng should take a quick look at the SQL changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167671951
Looks good.
---
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 featur
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10440#discussion_r48510554
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/Node.scala ---
@@ -386,9 +386,9 @@ private[tree] object LearningNode {
var levelsToGo = in
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10440#discussion_r48510515
--- Diff: launcher/src/main/java/org/apache/spark/launcher/Main.java ---
@@ -151,7 +151,7 @@ private static String
prepareWindowsCommand(List cmd, Map
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10499#issuecomment-167670013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10499#issuecomment-167670012
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10499#issuecomment-167669938
**[Test build #48373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48373/consoleFull)**
for PR 10499 at commit
[`7884e87`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9185#issuecomment-167668475
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9185#issuecomment-167668477
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167668461
**[Test build #48376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48376/consoleFull)**
for PR 10479 at commit
[`422ef49`](https://gi
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167668145
@andrewor14 could you take a look at this pr? 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
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10499#discussion_r48508973
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -119,7 +119,7 @@ final class DataFrameWriter private[sql](df: DataFrame
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10368#discussion_r48508806
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -109,6 +111,9 @@ class KryoSerializer(conf: SparkConf)
kryo.re
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9185#issuecomment-167665605
This seems fine to me as a first step. Eventually we will probably want to
make the RBackend multi-session aware.
---
If your project is set up for it, you can reply
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167665578
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10253
---
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 ena
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/10253#issuecomment-167665206
LGTM. Merging this into `master` and `branch-1.6`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user falaki commented on the pull request:
https://github.com/apache/spark/pull/9185#issuecomment-167664825
ping @marmbrus
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10438#issuecomment-167664384
**[Test build #48374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48374/consoleFull)**
for PR 10438 at commit
[`67611ac`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10498#issuecomment-167664157
BTW in github you can use square brackets to create a checklist, e.g.
```
- [] item a
- [] item b
```
becomes
- [] item a
- [] ite
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10471#discussion_r48507936
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala ---
@@ -102,6 +102,7 @@ case class SetCommand(kv: Option[(String,
Option[String
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/10368#discussion_r48507734
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -109,6 +111,9 @@ class KryoSerializer(conf: SparkConf)
kryo.regi
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10368#discussion_r48507496
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -109,6 +111,9 @@ class KryoSerializer(conf: SparkConf)
kryo.re
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10464#issuecomment-167661660
Looks a race condition in `restart` and `finally { ... socket.stop() ...}`.
`restart` will start a new thread and call `receiver.onStart`. So
`receiver.onStart` may run
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10459#discussion_r48505634
--- Diff: core/src/test/scala/org/apache/spark/util/AkkaUtilsSuite.scala ---
@@ -61,9 +55,14 @@ class AkkaUtilsSuite extends SparkFunSuite with
LocalSparkCon
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10459#discussion_r48505487
--- Diff: core/src/test/scala/org/apache/spark/util/AkkaUtilsSuite.scala ---
@@ -61,9 +55,14 @@ class AkkaUtilsSuite extends SparkFunSuite with
LocalSparkCon
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10440#issuecomment-167656781
ML changes look good to me. 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 d
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10499#issuecomment-167655997
**[Test build #48373 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48373/consoleFull)**
for PR 10499 at commit
[`7884e87`](https://gi
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/10475#issuecomment-167655015
CC: @harishreedharan
@SaintBacchus could you add test cases for this change?
---
If your project is set up for it, you can reply to this email and have your
r
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10459#issuecomment-167654637
**[Test build #48372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48372/consoleFull)**
for PR 10459 at commit
[`1f5a523`](https://gi
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/10487#issuecomment-167653173
@QiangCai LGTM but could you 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 pro
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10393
---
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 ena
GitHub user ted-yu opened a pull request:
https://github.com/apache/spark/pull/10499
[DOC] Adjust coverage for partitionBy()
This is the related thread:
http://search-hadoop.com/m/q3RTtO3ReeJ1iF02&subj=Re+partitioning+json+data+in+spark
Michael suggested fixing the doc.
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10393#issuecomment-167651518
Thanks, merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10174
---
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 ena
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10174#issuecomment-167650817
Thanks, merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user nongli commented on the pull request:
https://github.com/apache/spark/pull/10498#issuecomment-167649759
@cloud-fan
> currently we don't shuffle before writing partitioned data, which means
we will have same partition data in different RDD blocks, and that's why we
hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10481#issuecomment-167649006
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10481#issuecomment-167649007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167648846
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167648836
**[Test build #48370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48370/consoleFull)**
for PR 9608 at commit
[`946ec59`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167648843
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
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/10444#discussion_r48503899
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -47,6 +48,34 @@ trait Predicate extends Expression {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10381
---
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 ena
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10488#discussion_r48503757
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -57,9 +57,10 @@ case class Md5(child: Expression) extends
U
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/10381#issuecomment-167648268
Thanks @srowen , @BenFradet and @yanboliang for the review. I'm merging
this into `master` and `branch-1.6`.
---
If your project is set up for it, you can reply to th
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10390#issuecomment-167648261
Okay, I'm fine with merging this now to unblock other progress. However, I
think we should move the test to be with the other encoder tests.
---
If your project is s
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10327#discussion_r48503471
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/PCA.scala ---
@@ -167,14 +167,37 @@ object PCAModel extends MLReadable[PCAModel] {
pr
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10398
---
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 ena
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/8200#discussion_r48503308
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -116,13 +110,4 @@ class BooleanSimplific
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10398#issuecomment-167645851
Thanks, merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167645599
**[Test build #48370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48370/consoleFull)**
for PR 9608 at commit
[`946ec59`](https://gith
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10456
---
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 ena
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/10456#issuecomment-167645225
O.K, I'll merge this into `master` and `branch-1.6`. Thanks @wyaron !
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10327#issuecomment-167645124
My only comment is that, in general, I'd like us to add unit tests for
backwards compatibility (as was done for spark.mllib NaiveBayesModel). But I'm
OK with skippin
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/10327#discussion_r48502942
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/PCA.scala ---
@@ -167,14 +167,37 @@ object PCAModel extends MLReadable[PCAModel] {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167644604
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167644603
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 projec
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9321#issuecomment-167644002
> On the other hand, I cannot see serious issues by allowing user to
potentially read any files under workerDir, it will display the bytecode if
reading a jar file bu
Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/8200#discussion_r48502484
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -464,6 +465,24 @@ object OptimizeIn extends Rule[LogicalP
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167643902
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167643900
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 projec
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167643628
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9608#issuecomment-167643636
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/4399#issuecomment-167643533
Weird, something is wrong with this PR. Maybe re-open it and see if the
problem goes away?
---
If your project is set up for it, you can reply to this email and have
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10435#discussion_r48502287
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -176,3 +178,221 @@ case class Crc32(child: Expression) extend
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10456#issuecomment-167643456
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
enabl
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/4027#issuecomment-167643240
> It's much easier (for me) to think about the number of cores and amount
of memory per NODE
You did not understand my suggestion. That's exactly what I'm sug
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10388
---
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 ena
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10388#issuecomment-167642446
Thanks, merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167641066
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167640841
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
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10443#discussion_r48501675
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects.scala
---
@@ -241,7 +241,7 @@ case class NewInstance(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10451#discussion_r48501690
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala
---
@@ -448,7 +448,21 @@ case class Pivot(
Github user wilson8 commented on the pull request:
https://github.com/apache/spark/pull/10486#issuecomment-167640827
@vectorijk
Thanks. I changed the commit message.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10391#discussion_r48501635
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects.scala
---
@@ -439,6 +439,13 @@ case class MapObjects(
s"
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10479#issuecomment-167639232
I'm only planning to merge this into 2.0.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 pr
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10451#discussion_r48501449
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala
---
@@ -448,7 +448,21 @@ case class Pivot(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10494#discussion_r48501359
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LocalRelation.scala
---
@@ -62,6 +62,10 @@ case class LocalRelation(out
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10479#discussion_r48501316
--- Diff:
extras/kinesis-asl/src/main/scala/org/apache/spark/streaming/kinesis/KinesisInputDStream.scala
---
@@ -30,7 +30,7 @@ import
org.apache.spark.st
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10430
---
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 ena
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10479#discussion_r48501332
--- Diff:
extras/kinesis-asl/src/main/scala/org/apache/spark/streaming/kinesis/KinesisBackedBlockRDD.scala
---
@@ -70,12 +70,12 @@ class KinesisBackedBloc
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10479#discussion_r48501272
--- Diff:
extras/kinesis-asl/src/main/scala/org/apache/spark/streaming/kinesis/KinesisBackedBlockRDD.scala
---
@@ -70,12 +70,12 @@ class KinesisBackedBloc
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10430#issuecomment-167638369
Thanks, merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not hav
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10435#discussion_r48500881
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -176,3 +178,221 @@ case class Crc32(child: Expression) ex
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/10488#discussion_r48500799
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -57,9 +57,10 @@ case class Md5(child: Expression) extends
U
201 - 300 of 418 matches
Mail list logo
