Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219932692
O.K. Initially we add some characters to the whitelist and if we need some
more characters, we'll consider whether it should be add or not at any time.
How about this
Github user taku-k commented on the pull request:
https://github.com/apache/spark/pull/12767#issuecomment-219938151
@MLnick Thanks! My JIRA username is `taku-k`. It's the same as the github
username.
---
If your project is set up for it, you can reply to this email and have your
Github user ueshin commented on the pull request:
https://github.com/apache/spark/pull/13147#issuecomment-219938062
@marmbrus Thank you for your comments.
I see but apparently we are lacking the `planLater` functionality for extra
strategies.
Could you give me the earlier
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13162#issuecomment-219942612
Seem like a good idea to me. cc @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
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/13164
[SPARK-15031][EXAMPLE] Use SparkSession in examples
## What changes were proposed in this pull request?
Use `SparkSession` according to
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219935334
Either way works for me.
---
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 MLnick commented on the pull request:
https://github.com/apache/spark/pull/12767#issuecomment-219937205
@taku-k please provide your JIRA username so I can assign the ticket to you.
---
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/12767
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13163#issuecomment-219944557
**[Test build #58752 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58752/consoleFull)**
for PR 13163 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13104#issuecomment-219950378
**[Test build #58757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58757/consoleFull)**
for PR 13104 at commit
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/13163
[SPARK-15360][Spark-Submit]Should print spark-submit usage when no
arguments is specified
## What changes were proposed in this pull request?
(Please fill in changes proposed in this
Github user jurriaan commented on the pull request:
https://github.com/apache/spark/pull/13104#issuecomment-219950253
@rxin changed the directory name to text-partitioned
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13164#issuecomment-219950369
**[Test build #58756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58756/consoleFull)**
for PR 13164 at commit
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/12767#issuecomment-219936689
Merged to master/branch-2.0. Thanks @vectorijk @holdenk @sethah for the
review!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13155#issuecomment-219936663
**[Test build #2990 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2990/consoleFull)**
for PR 13155 at commit
Github user jurriaan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13104#discussion_r63655222
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala
---
@@ -65,6 +65,14 @@ class TextSuite extends QueryTest
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/13162#issuecomment-219943826
I agree this would be useful for debugging. Can we plug it in as a
`StreamSinkProvider` instead of hard coding an `if` into the `DataSource`
resolver?
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-219947527
**[Test build #58746 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58746/consoleFull)**
for PR 12979 at commit
Github user tejasapatil commented on the pull request:
https://github.com/apache/spark/pull/12194#issuecomment-219941867
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
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219948559
LGTM, merging to master and branch-2.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
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219934463
@sarutak That's true. Should `/` also be common used?
---
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 sarutak commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219944159
I have one concern about the whitelisting approach. Even if each single
character is safe, it's difficult to ensure any character sequences which
consist of those safe
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219946313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219946121
**[Test build #58748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58748/consoleFull)**
for PR 13127 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-219947764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12979#issuecomment-219947763
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219933026
**[Test build #58747 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58747/consoleFull)**
for PR 13127 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219933074
@rxin If there is any new bug found on this, we could switch to white list,
otherwise I'd like to have the current solution.
---
If your project is set up for it, you
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219935041
**[Test build #58748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58748/consoleFull)**
for PR 13127 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-219938017
**[Test build #58749 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58749/consoleFull)**
for PR 10125 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219940081
**[Test build #58747 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58747/consoleFull)**
for PR 13127 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219940202
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13127#issuecomment-219940203
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12194#issuecomment-219942382
**[Test build #58750 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58750/consoleFull)**
for PR 12194 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219945292
hm that's true. @davies want to review that one?
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13160#issuecomment-219946772
**[Test build #58755 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58755/consoleFull)**
for PR 13160 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13160#issuecomment-219946774
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13160#issuecomment-219946777
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219933348
@davies this is the 2nd security bug with codegen we found already.
@sarutak sgtm.
---
If your project is set up for it, you can reply to this email and have
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/12939#issuecomment-219938986
`/` is used as the division operator and `*` is used as the multiplication
operator so it's good to add those characters but we should `*/` so we need to
add `\*(?!/)`
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13163#issuecomment-219943467
**[Test build #58751 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58751/consoleFull)**
for PR 13163 at commit
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-219945586
I just did rebase to solve the conflict.
---
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 kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-219945597
retest it 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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10125#issuecomment-219945609
**[Test build #58754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58754/consoleFull)**
for PR 10125 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13104#discussion_r63655817
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala
---
@@ -65,6 +65,14 @@ class TextSuite extends QueryTest
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220138483
**[Test build #58804 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58804/consoleFull)**
for PR 13158 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220138708
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220145888
**[Test build #2993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2993/consoleFull)**
for PR 13158 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13156#discussion_r63757822
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ddl.scala ---
@@ -126,24 +126,9 @@ case class RefreshTable(tableIdent:
Github user frreiss commented on a diff in the pull request:
https://github.com/apache/spark/pull/13155#discussion_r63759450
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1648,16 +1648,56 @@ object
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/13037#issuecomment-220121435
"Locality level summary" doesn't have any new lines, but it can have up to
four string: int pairs separated by a semi-colon so it may line wrap.
---
If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220132475
**[Test build #58797 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58797/consoleFull)**
for PR 8559 at commit
Github user frreiss commented on a diff in the pull request:
https://github.com/apache/spark/pull/13155#discussion_r63767862
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -293,4 +293,65 @@ class SubquerySuite extends QueryTest with
Github user frreiss commented on a diff in the pull request:
https://github.com/apache/spark/pull/13155#discussion_r63767826
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -293,4 +293,65 @@ class SubquerySuite extends QueryTest with
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220137024
**[Test build #58800 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58800/consoleFull)**
for PR 13158 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220137257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220137256
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13161#issuecomment-220139382
Yea this should not go into 2.0.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13175#issuecomment-220143489
**[Test build #58806 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58806/consoleFull)**
for PR 13175 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220144611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220144607
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13156#issuecomment-220149660
**[Test build #58805 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58805/consoleFull)**
for PR 13156 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13156#discussion_r63758414
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SessionState.scala ---
@@ -163,6 +163,9 @@ private[sql] class SessionState(sparkSession:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13175#issuecomment-220120542
**[Test build #58806 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58806/consoleFull)**
for PR 13175 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12836#issuecomment-220120530
**[Test build #58807 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58807/consoleFull)**
for PR 12836 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13176#issuecomment-220125864
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220132741
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220132746
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/13174#issuecomment-220133322
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13174#issuecomment-220133615
**[Test build #58811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58811/consoleFull)**
for PR 13174 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13120#discussion_r63772240
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -348,13 +349,23 @@ object
Github user wangyang1992 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13177#discussion_r63773414
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -480,7 +480,7 @@ private[client] class Shim_v0_13 extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13175#issuecomment-220143803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13175#issuecomment-220143799
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13045#issuecomment-220147655
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user ericl commented on the pull request:
https://github.com/apache/spark/pull/13161#issuecomment-220148117
@davies @rxin sounds good
I also added a before/after result of sorting nullable longs in the PR
description.
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220147479
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13174#issuecomment-220128440
**[Test build #58796 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58796/consoleFull)**
for PR 13174 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13045#issuecomment-220128385
**[Test build #58798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58798/consoleFull)**
for PR 13045 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220131079
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13174#issuecomment-220128685
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13045#issuecomment-220128719
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/12979#discussion_r63763448
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala
---
@@ -133,7 +133,9 @@ object
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220130869
**[Test build #58799 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58799/consoleFull)**
for PR 8559 at commit
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/12762#issuecomment-220129049
Merged to master/branch-2.0
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13045#issuecomment-220128712
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13163#issuecomment-220129969
**[Test build #58810 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58810/consoleFull)**
for PR 13163 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-220131077
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12762
---
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 dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12719#issuecomment-220134254
In addition, sorry for that I wrote a wrong example before
`checkEvaluation(FormatNumber(Literal(4.asInstanceOf[Byte]), Literal(3)),
"4.000")`.
---
If your
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/13174#issuecomment-220138657
So, this branch passes all tests for me locally. If it still faces some
spurious failures, I'm going to merge it anyway.
---
If your project is set up for it, you can
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13155#discussion_r63770055
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1648,16 +1648,56 @@ object
Github user frreiss commented on a diff in the pull request:
https://github.com/apache/spark/pull/13155#discussion_r63768232
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1648,16 +1648,56 @@ object
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220138706
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13156#issuecomment-220149993
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13156#issuecomment-220149990
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13158#issuecomment-220147151
**[Test build #58803 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58803/consoleFull)**
for PR 13158 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13045#issuecomment-220147299
**[Test build #58809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58809/consoleFull)**
for PR 13045 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13156#issuecomment-220117679
**[Test build #58805 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58805/consoleFull)**
for PR 13156 at commit
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/13159#discussion_r63758013
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/PCA.scala ---
@@ -53,7 +53,8 @@ private[feature] trait PCAParams extends Params with
1 - 100 of 862 matches
Mail list logo