Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107337310
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107335518
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17381
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17381#discussion_r107335448
--- Diff: sql/core/src/test/resources/sql-tests/results/describe.sql.out ---
@@ -79,30 +92,114 @@ d string
-- !query
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107335421
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17381#discussion_r107335417
--- Diff: sql/core/src/test/resources/sql-tests/results/describe.sql.out ---
@@ -79,30 +92,114 @@ d string
-- !query
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107335293
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75020/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13932
**[Test build #75020 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75020/testReport)**
for PR 13932 at commit
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/17308
@tcondie and @zsxwing any comments on this patch. I would be happy, if this
bug is fixed before 2.2 is released.
---
If your project is set up for it, you can reply to this email and have your
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/17358
Actually thank you for your comments, I missed one file in swarm of errors
from generated files.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17358
Sorry for making this PR dragged. I am fine. Let me cope with it separately.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17358
**[Test build #75028 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75028/testReport)**
for PR 17358 at commit
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/17358
@HyukjinKwon, Fixing those errors require fixing unidoc plugin's java code
generation. IMO, that has a broader scope. What do you think ?
---
If your project is set up for it, you can reply to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17251
**[Test build #75027 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75027/testReport)**
for PR 17251 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17251
Retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17363
Hi, @vanzin .
Could you review this backport when you have some time?
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17274
---
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/17274
Merged into master, I'd prepare a separate PR for branch-2.1. Thanks for
reviewing.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r107332590
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -61,6 +63,37 @@ abstract class
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/17274
@felixcheung Yeah, this PR not aim to make users can call SparkR package
inside UDF, it only focus on users can call ```spark.getSparkFiles``` on
workers. I don't think we should support using
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r107331403
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -61,6 +63,37 @@ abstract class SubqueryExpression(
Github user weiqingy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r107331216
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -148,6 +149,8 @@ private[sql] class SharedState(val sparkContext:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17361
---
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 brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17250#discussion_r107330754
--- Diff:
external/kinesis-asl/src/main/scala/org/apache/spark/streaming/kinesis/SerializableCredentialsProvider.scala
---
@@ -83,3 +84,146 @@
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17348
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17348
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75019/
Test PASSed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107330581
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17348
**[Test build #75019 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75019/testReport)**
for PR 17348 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17381
**[Test build #75026 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75026/testReport)**
for PR 17381 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107329860
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala ---
@@ -230,4 +230,17 @@ class DataFramePivotSuite extends QueryTest
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107329634
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16596
Thanks for cc'ing me. I just ran a build against branch-2.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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17361
**[Test build #3604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3604/testReport)**
for PR 17361 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16596
Build started: [SparkR][BRANCH-2.1]
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17348
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75018/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17348
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107329376
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107329399
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17348
**[Test build #75018 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75018/testReport)**
for PR 17348 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107328723
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107328381
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17300
The fix LGTM, I think it is nice to have such topology priority. CC
@mridulm .
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17381#discussion_r107327958
--- Diff: sql/core/src/test/resources/sql-tests/inputs/describe.sql ---
@@ -8,6 +8,10 @@ DESC t;
DESC TABLE t;
+DESC FORMATTED t;
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17381#discussion_r107327852
--- Diff: sql/core/src/test/resources/sql-tests/inputs/describe.sql ---
@@ -8,6 +8,10 @@ DESC t;
DESC TABLE t;
+DESC FORMATTED t;
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17100
To detect duplicate names, you need to pass SQLConf `conf` in
`checkAnalysis` for supporting the case sensitivity in name resolution. For
each missing attribute, you can try using
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17381#discussion_r107327601
--- Diff: sql/core/src/test/resources/sql-tests/inputs/describe.sql ---
@@ -8,6 +8,10 @@ DESC t;
DESC TABLE t;
+DESC FORMATTED t;
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17355
**[Test build #75025 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75025/testReport)**
for PR 17355 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107327362
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -56,6 +57,44 @@ private[spark] class BlockResult(
val bytes:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107327188
--- Diff: core/src/main/scala/org/apache/spark/storage/DiskStore.scala ---
@@ -17,48 +17,67 @@
package org.apache.spark.storage
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107326950
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -486,14 +486,16 @@ class Analyzer(
case
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17342
CC @vanzin @tgravescs , can you please also review 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 your project
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107326715
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1065,7 +1084,7 @@ private[spark] class BlockManager(
try
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75022/
Test FAILed.
---
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r107326694
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -148,6 +149,8 @@ private[sql] class SharedState(val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #75022 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75022/testReport)**
for PR 14617 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17361
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75017/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17361
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15435
**[Test build #75024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75024/testReport)**
for PR 15435 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17361
**[Test build #75017 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75017/testReport)**
for PR 17361 at commit
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15435
@sethah Code updated.
cc @yanboliang Pls help boost 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 your
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15435#discussion_r107326057
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -1786,51 +1793,98 @@ class
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15435#discussion_r107325971
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -194,15 +207,9 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #75023 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75023/testReport)**
for PR 14617 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r107325822
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -337,11 +337,29 @@ trait CheckAnalysis extends
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17348#discussion_r107325633
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -486,14 +486,16 @@ class Analyzer(
case
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #75022 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75022/testReport)**
for PR 14617 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75021/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #75021 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75021/testReport)**
for PR 14617 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/14617
Hi All, thanks a lot for your comments.
Here is the UI after changed:
![screen shot 2017-03-22 at 11 10 22
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17100
Can you remove `[PYTHON]` from the PR title? 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107324613
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerManagedBuffer.scala ---
@@ -31,17 +35,31 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107324480
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerManagedBuffer.scala ---
@@ -31,17 +35,31 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107324269
--- Diff:
core/src/main/scala/org/apache/spark/storage/BlockManagerManagedBuffer.scala ---
@@ -17,11 +17,15 @@
package
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #75021 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75021/testReport)**
for PR 14617 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107323983
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -56,6 +57,44 @@ private[spark] class BlockResult(
val bytes:
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/17014
ping @hhbyyh
I updated the PR, can you please help reviewing this? Thank in advance.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107323833
--- Diff:
core/src/main/scala/org/apache/spark/serializer/SerializerManager.scala ---
@@ -167,30 +167,26 @@ private[spark] class SerializerManager(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107323552
--- Diff:
core/src/main/scala/org/apache/spark/security/CryptoStreamUtils.scala ---
@@ -48,12 +51,30 @@ private[spark] object CryptoStreamUtils extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75016/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #75016 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75016/testReport)**
for PR 15628 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107323246
--- Diff:
core/src/main/scala/org/apache/spark/security/CryptoStreamUtils.scala ---
@@ -63,12 +84,27 @@ private[spark] object CryptoStreamUtils extends
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107323253
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameImplicitsSuite.scala ---
@@ -51,4 +51,15 @@ class DataFrameImplicitsSuite extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107323158
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107323085
--- Diff:
core/src/main/scala/org/apache/spark/security/CryptoStreamUtils.scala ---
@@ -48,12 +51,30 @@ private[spark] object CryptoStreamUtils extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17295#discussion_r107322905
--- Diff:
core/src/main/scala/org/apache/spark/broadcast/TorrentBroadcast.scala ---
@@ -219,18 +219,22 @@ private[spark] class TorrentBroadcast[T:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17378
cc @hvanhovell
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17358
If my suggestion is the reason to hold off this PR currently and you are
not sure of it, I am fine. I can double-check and deal with this in a separate
PR.
---
If your project is set up for
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r107321520
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -41,7 +41,20 @@ object CatalystSerde {
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13932
**[Test build #75020 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75020/testReport)**
for PR 13932 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13932
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17350
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17379
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user kunalkhamar commented on the issue:
https://github.com/apache/spark/pull/17382
@tdas all 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. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17379
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75015/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17382
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
1 - 100 of 703 matches
Mail list logo