Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17315
For me it looks good as targrted.
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17339
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 windpiger commented on the issue:
https://github.com/apache/spark/pull/17338
@gatorsmile sbt test succeed, and mvn `create temporary view using` 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
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17339
test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r106866491
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -61,6 +63,37 @@ abstract class SubqueryExpression(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17311
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/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 yanboliang opened a pull request:
https://github.com/apache/spark/pull/17360
[WIP][SPARK-20029][ML] ML LinearRegression supports bound constrained
optimization.
## What changes were proposed in this pull request?
MLlib ```LinearRegression``` should support bound
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
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/16867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74870/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #74870 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74870/testReport)**
for PR 16867 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17355
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/17355
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74871/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17355
**[Test build #74871 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74871/testReport)**
for PR 17355 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
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/17338
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74869/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17338
**[Test build #74869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74869/testReport)**
for PR 17338 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17315
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74867/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17315
Build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17315
**[Test build #74867 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74867/testReport)**
for PR 17315 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74865/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
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/17354
**[Test build #74865 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74865/testReport)**
for PR 17354 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17315
Sorry, my PR got merged and made a conflict with this. These are all from
me. Probably, let me try to leave some more trivial comments while it's open.
---
If your project is set up for it,
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106852242
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -233,81 +187,39 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17334
**[Test build #74872 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74872/testReport)**
for PR 17334 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17329#discussion_r106848680
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/buffer/FileSegmentManagedBuffer.java
---
@@ -37,13 +37,24 @@
* A {@link
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17334
Jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17302#discussion_r106847852
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExistingRDD.scala ---
@@ -70,7 +70,20 @@ object RDDConversions {
object ExternalRDD
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106847502
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -113,8 +113,11 @@ class CSVFileFormat
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106847415
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -55,108 +52,6 @@ class JacksonParser(
private
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17314#discussion_r106847289
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/r/AFTSurvivalRegressionWrapper.scala
---
@@ -30,6 +32,7 @@ import
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17358#discussion_r106847044
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/crypto/ClientChallenge.java
---
@@ -28,7 +28,7 @@
/**
* The client
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15363
thanks, merging to master!
The nest step is consolidating this with CBO, looking forward to it :)
---
If your project is set up for it, you can reply to this email and have your
reply
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15363
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106846685
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/StarJoinReorderSuite.scala
---
@@ -0,0 +1,580 @@
+/*
+ * Licensed
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106846617
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/StarJoinReorderSuite.scala
---
@@ -0,0 +1,580 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
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/17354
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74868/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17354
**[Test build #74868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74868/testReport)**
for PR 17354 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
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/17338
**[Test build #74866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74866/testReport)**
for PR 17338 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74866/
Test FAILed.
---
Github user gczsjdy commented on the issue:
https://github.com/apache/spark/pull/17359
cc @chenghao-intel @yucai @adrian-wang @cloud-fan @gatorsmile
---
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 ehsun7b commented on a diff in the pull request:
https://github.com/apache/spark/pull/17314#discussion_r106844740
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/r/AFTSurvivalRegressionWrapper.scala
---
@@ -30,6 +32,7 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17359
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user gczsjdy opened a pull request:
https://github.com/apache/spark/pull/17359
Add aggreagate expression nGrams
## What changes were proposed in this pull request?
This is the implementation of Hive's `ngrams`, which is a popular
statistical and data mining
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17342
Can you update the title to add `[SQL]` module? It looks like a sql
specific problem.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17355
**[Test build #74871 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74871/testReport)**
for PR 17355 at commit
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17353#discussion_r106843645
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -202,14 +201,15 @@ object JoinReorderDP
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17353#discussion_r106843609
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -202,14 +201,15 @@ object JoinReorderDP
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17355
Jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17353
Where should we check the count? Whom do we want to expose it to?
How about a debug level log?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #74870 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74870/testReport)**
for PR 16867 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17354
**[Test build #74868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74868/testReport)**
for PR 17354 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17338
**[Test build #74869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74869/testReport)**
for PR 17338 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16596
Jenkins, AppVeyor are both passing.
Tested manually on Windows - this change matches the behavior on *nix
---
If your project is set up for it, you can reply to this email and have your
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r106841997
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -148,6 +149,8 @@ private[sql] class SharedState(val
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r106841824
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2767,3 +2767,24 @@ private[spark] class CircularBuffer(sizeInBytes: Int
= 10240)
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17342#discussion_r106842764
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2767,3 +2767,24 @@ private[spark] class CircularBuffer(sizeInBytes: Int
= 10240)
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17338
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/17354
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 a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106842646
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -113,8 +113,11 @@ class CSVFileFormat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17338
**[Test build #74866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74866/testReport)**
for PR 17338 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17315
**[Test build #74867 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74867/testReport)**
for PR 17315 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17354
**[Test build #74865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74865/testReport)**
for PR 17354 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17315
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/17354
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/17338
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17355
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74853/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17355
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/17315
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/17315
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74857/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
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/17357
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74862/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17353
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74864/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
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/17358
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/16867
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/17353
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/17357
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/17354
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74859/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74856/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74858/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17358
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74863/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17353#discussion_r106841610
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -202,14 +201,15 @@ object JoinReorderDP
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17353#discussion_r106841523
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -202,14 +201,15 @@ object JoinReorderDP
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/17354
@gatorsmile Thanks a lot.
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17353#discussion_r106841464
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -202,14 +201,15 @@ object JoinReorderDP
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74852/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
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/17052
**[Test build #74852 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74852/testReport)**
for PR 17052 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17338
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/17338
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74855/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17338
**[Test build #74855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74855/testReport)**
for PR 17338 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17358
Hi @ScrapCodes, I just happened to look at this PR and ran `jeykill build`
against this PR.
It seems there are few more instances to make javadoc8 working as below:
```
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106840993
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,340 @@ package
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17354
Yes, we can revert it back
---
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 dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/17354
@gatorsmile Hi Sean, with this fix, do we still require the other fix in
OrcSourceSuite ? We know that it can't hurt.. but it helps expose problems like
this ?
---
If your project is set up
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17353
**[Test build #74864 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74864/testReport)**
for PR 17353 at commit
501 - 600 of 635 matches
Mail list logo