Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10170#issuecomment-162441738
I think that the test failures in `BlockManagerSuite` are related to the
semantics of `maxNumBytesToFree` in `StaticMemoryManager.acquireUnrollMemory`.
[Prev
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9565#issuecomment-162441586
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9565#issuecomment-162441582
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 zjffdu opened a pull request:
https://github.com/apache/spark/pull/10172
[SPARK-12166][TEST] Unset hadoop related environment in testing
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zjffdu/spark SPARK-12166
Altern
Github user taishi-oss commented on the pull request:
https://github.com/apache/spark/pull/8534#issuecomment-162440255
I modified the code that was pointed out.
@sarutak @yu-iskw Thank you for reviewing!
---
If your project is set up for it, you can reply to this email and have yo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8534#issuecomment-162440197
**[Test build #47260 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47260/consoleFull)**
for PR 8534 at commit
[`6831e7a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10030#issuecomment-162439940
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/10030#issuecomment-162439938
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/9565#issuecomment-162439812
**[Test build #47259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47259/consoleFull)**
for PR 9565 at commit
[`f806755`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10030#issuecomment-162439769
**[Test build #47257 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47257/consoleFull)**
for PR 10030 at commit
[`37d8080`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10030#issuecomment-162437605
**[Test build #47257 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47257/consoleFull)**
for PR 10030 at commit
[`37d8080`](https://gi
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/10094#discussion_r46792542
--- Diff: R/pkg/R/window.R ---
@@ -0,0 +1,88 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license ag
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9963#issuecomment-162437086
**[Test build #47256 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47256/consoleFull)**
for PR 9963 at commit
[`0e1d796`](https://gith
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/10132#issuecomment-162437123
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 sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/10094#discussion_r46792190
--- Diff: R/pkg/R/window.R ---
@@ -0,0 +1,88 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license ag
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/9963#issuecomment-162436010
Hi @vanzin , from my test and understanding so far, I think calling
`reset()` on the first registration should be OK. But here I still don't change
to that way, for me
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/10030#issuecomment-162435647
@shivaram, rebased again. I can successfully rebase it to master on my dev
machine, don't know any other merge conflicts. Could you merge it before
merging any other PR
Github user watermen commented on the pull request:
https://github.com/apache/spark/pull/10169#issuecomment-162435295
@dilipbiswal Yes, SPARK-11246 has already fixed it, Thanks and I'll close
this PR.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user watermen closed the pull request at:
https://github.com/apache/spark/pull/10169
---
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 e
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162431824
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
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162428939
@shivaram @felixcheung @sun-rui Thank you everyone! Hopefully, my code
changes resolve all your concerns. I learned a lot from you! : )
---
If your project is set u
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162427732
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/10160#issuecomment-162427729
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/10160#issuecomment-162427500
**[Test build #47255 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47255/consoleFull)**
for PR 10160 at commit
[`a78109e`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162424451
**[Test build #47255 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47255/consoleFull)**
for PR 10160 at commit
[`a78109e`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46789803
--- Diff: R/pkg/R/DataFrame.R ---
@@ -692,8 +696,8 @@ setMethod("sample",
setMethod("sample_frac",
signature(x = "DataFrame", withRepla
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46789542
--- Diff: R/pkg/R/DataFrame.R ---
@@ -692,8 +696,8 @@ setMethod("sample",
setMethod("sample_frac",
signature(x = "DataFrame", withRepla
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162419713
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/10160#issuecomment-162419711
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/10160#issuecomment-162419603
**[Test build #47254 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47254/consoleFull)**
for PR 10160 at commit
[`4337c35`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-16241
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-162418828
**[Test build #47249 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47249/consoleFull)**
for PR 10058 at commit
[`122d3f2`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-162418887
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
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46789026
--- Diff: R/pkg/R/DataFrame.R ---
@@ -692,8 +696,8 @@ setMethod("sample",
setMethod("sample_frac",
signature(x = "DataFrame", withReplacem
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10167#issuecomment-162418221
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10167#issuecomment-162418222
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10167#issuecomment-162418159
**[Test build #47248 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47248/consoleFull)**
for PR 10167 at commit
[`c100251`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162418157
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/10160#issuecomment-162418156
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/10160#issuecomment-162418099
**[Test build #47253 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47253/consoleFull)**
for PR 10160 at commit
[`34d0118`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10170#issuecomment-162417990
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10170#issuecomment-162417956
**[Test build #47251 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47251/consoleFull)**
for PR 10170 at commit
[`0eac7da`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10170#issuecomment-162417991
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162417739
**[Test build #47254 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47254/consoleFull)**
for PR 10160 at commit
[`4337c35`](https://gi
Github user guoxu1231 commented on the pull request:
https://github.com/apache/spark/pull/8934#issuecomment-162417597
BTW, There is a JIRA to suggest user to prefer using c['column'] instead of
c.column.
@davies, we encountered similar issue, could you paste the JIRA number fo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10169#issuecomment-162417059
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/10169#issuecomment-162417058
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 jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/9963#discussion_r46788434
--- Diff:
yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -51,6 +51,9 @@ private[spark] abstract class YarnSchedul
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10169#issuecomment-162417004
**[Test build #47250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47250/consoleFull)**
for PR 10169 at commit
[`f1ef856`](https://g
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9988#issuecomment-162416976
LGTM except a nit
---
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 feat
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162416996
: ) @sun-rui Done. Thank you!
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46788421
--- Diff: R/pkg/R/DataFrame.R ---
@@ -677,13 +677,15 @@ setMethod("unique",
#' collect(sample(df, TRUE, 0.5))
#'}
setMethod("sample",
-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10171#issuecomment-162416419
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10171#issuecomment-162416422
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10171#issuecomment-162416192
**[Test build #47252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47252/consoleFull)**
for PR 10171 at commit
[`7c7c5d3`](https://g
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46788139
--- Diff: R/pkg/R/DataFrame.R ---
@@ -677,13 +677,15 @@ setMethod("unique",
#' collect(sample(df, TRUE, 0.5))
#'}
setMethod("sample",
-
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/10157#discussion_r46788170
--- Diff:
core/src/main/scala/org/apache/spark/deploy/SparkSubmitArguments.scala ---
@@ -177,7 +177,7 @@ private[deploy] class SparkSubmitArguments(args:
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/10160#discussion_r46788107
--- Diff: R/pkg/R/DataFrame.R ---
@@ -692,8 +694,12 @@ setMethod("sample",
setMethod("sample_frac",
signature(x = "DataFrame", withReplace
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162415989
**[Test build #47253 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47253/consoleFull)**
for PR 10160 at commit
[`34d0118`](https://gi
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162415645
LGTM. I'll wait for @felixcheung and Jenkins before merging
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/10157#issuecomment-162415650
Looks like this is a hidden environment, do we need to support it
explicitly?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162415274
@felixcheung @shivaram Sure, just added that test case. Please review it.
Thank you! : )
---
If your project is set up for it, you can reply to this email and have
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9988#discussion_r46787805
--- Diff: streaming/src/test/resources/log4j.properties ---
@@ -25,4 +25,5 @@ log4j.appender.file.layout.ConversionPattern=%d{yy/MM/dd
HH:mm:ss.SSS} %t %p %c{
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10110#issuecomment-162414966
LGTM for the 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 f
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10110#discussion_r46787547
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/BatchedWriteAheadLog.scala
---
@@ -165,10 +165,12 @@ private[util] class BatchedWriteAhea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10171#issuecomment-162414262
**[Test build #47252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47252/consoleFull)**
for PR 10171 at commit
[`7c7c5d3`](https://gi
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162413477
Yeah thats a good idea @felixcheung
---
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 do
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/10171
[SPARK-12168][SPARKR] Add automated tests for conflicted function in R
Currently this is reported when loading the SparkR package in R (probably
would add is.nan)
```
Loading required p
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/10160#issuecomment-162412228
@gatorsmile Sure - I guess the main thing is to ensure the seed is getting
set. How about:
```
count1 <- count(sample(df, FALSE, 0.1, 0))
count2 <- cou
Github user dilipbiswal commented on the pull request:
https://github.com/apache/spark/pull/10169#issuecomment-162407562
@watermen I believe SPARK-11246 was a very similar defect and was fixed in
1.5. Is this a different scenario that what was addressed in that defect ?
@yhuai had
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/9490#issuecomment-162406408
@yhuai Sure, np
---
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/10170#issuecomment-162405970
**[Test build #47251 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47251/consoleFull)**
for PR 10170 at commit
[`0eac7da`](https://gi
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/10170
[SPARK-12165] Fix bug in eviction of storage memory by execution
This patch fixes a bug in the eviction of storage memory by execution.
## The bug:
In general, execution should b
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10169#issuecomment-162404057
**[Test build #47250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47250/consoleFull)**
for PR 10169 at commit
[`f1ef856`](https://gi
GitHub user watermen opened a pull request:
https://github.com/apache/spark/pull/10169
[SPARK-12167][SQL] Invoke the right sameResult function when plan is
warpped with SubQueries
### Bug
I find this bug when I use cache table,
```
spark-sql> create table src_p(key int,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/9565#discussion_r46785072
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -66,980 +85,305 @@ case class ScalaUDF(
*/
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/9490#issuecomment-162402641
@zjffdu How about we revisit it after we release 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 yo
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/9490#issuecomment-162402210
@yhuai @liancheng any more comments ?
---
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 microwishing closed the pull request at:
https://github.com/apache/spark/pull/10168
---
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
GitHub user microwishing opened a pull request:
https://github.com/apache/spark/pull/10168
Fix kstreaming war
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/microwishing/spark FIX_KSTREAMING_WAR
Alternatively you can review and
Github user microwishing commented on the pull request:
https://github.com/apache/spark/pull/10168#issuecomment-162402045
wrong PR. sorry
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/9565#discussion_r46784210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -66,980 +85,305 @@ case class ScalaUDF(
*/
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/10165#issuecomment-162400140
I have the exact same question when calling the show function. From the
perspectives of users, they might not care the encoded values at all when
calling the functio
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-162397994
**[Test build #47249 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47249/consoleFull)**
for PR 10058 at commit
[`122d3f2`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10163#issuecomment-162397946
This is a great feature! Can we implement it in individual expressions
instead of centralizing them in `LogicalPlan.samResult`? A lof of commutative
operators need it
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10167#issuecomment-162397948
**[Test build #47248 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47248/consoleFull)**
for PR 10167 at commit
[`c100251`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10165#issuecomment-162396682
Shoud we print the decoded values(user objects) in `Dataset.show`? cc
@marmbrus @rxin
---
If your project is set up for it, you can reply to this email and have you
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/10167
[SPARK-12060][CORE] Avoid memory copy in JavaSerializerInstance.serialize
Merged #10051 again since #10083 is resolved.
This reverts commit 328b757d5d4486ea3c2e246780792d7a57ee85e5.
You ca
Github user lianhuiwang commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-162396051
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 thi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10141#discussion_r46781460
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala ---
@@ -40,7 +40,8 @@ import org.apache.spark.sql.catalyst.util.DataTypeParser
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10151
---
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 holdenk commented on the pull request:
https://github.com/apache/spark/pull/10166#issuecomment-162386176
It might be useful to also document the different ways "missing" labels can
be handled - what are your thoughts?
---
If your project is set up for it, you can reply to
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10151#issuecomment-162386145
We can handle retention of the output in a followup, since it'd probably be
best to also do it for Java, Python, and R style checks. Therefore, I'm going
to merge thi
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/10037#issuecomment-162385822
Thanks for your help @sun-rui @felixcheung @shivaram .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/10154#discussion_r46776484
--- Diff:
core/src/test/resources/HistoryServerExpectations/executor_list_json_expectation.json
---
@@ -4,11 +4,13 @@
"rddBlocks" : 8,
"memory
Github user dilipbiswal commented on the pull request:
https://github.com/apache/spark/pull/10156#issuecomment-162349876
@yhuai Hi Yin, Thanks for your comments. In this fix, we are only changing
the nullability of the value for comparision purpose to see if the left and
right types a
Github user karlhigley closed the pull request at:
https://github.com/apache/spark/pull/9843
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10161#issuecomment-162344223
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/10165#issuecomment-162344227
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/10165#issuecomment-162344226
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
1 - 100 of 176 matches
Mail list logo