Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134449445
[Test build #41505 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41505/console)
for PR 8007 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8404#issuecomment-134449415
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/8383#issuecomment-134449165
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8404#issuecomment-134449365
[Test build #41493 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41493/console)
for PR 8404 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8405#issuecomment-134453147
Merged build triggered.
---
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/8340#issuecomment-134453479
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/8405#issuecomment-134454028
[Test build #41507 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41507/consoleFull)
for PR 8405 at commit
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134462128
Mark it as WIP, not sure if @davies @rxin has better idea to fix it.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37829097
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user MechCoder commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37829471
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8411#issuecomment-134469559
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 liyezhang556520 opened a pull request:
https://github.com/apache/spark/pull/8412
[DOC] add missing parameters for scala doc
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/liyezhang556520/spark minorDoc
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134469987
Merged build triggered.
---
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/8412#issuecomment-134469992
Merged build started.
---
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/8412#issuecomment-134469983
Merged build triggered.
---
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/8007#issuecomment-134470022
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8391#discussion_r37833289
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -275,6 +275,7 @@ private[sql] class JDBCRDD(
*/
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134477474
Yes, the CheckOverflow(xxx, DecimalType(38,38)) will yield null.
---
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 pull request:
https://github.com/apache/spark/pull/8413#issuecomment-13440
[Test build #41519 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41519/consoleFull)
for PR 8413 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134477053
[Test build #41518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41518/consoleFull)
for PR 8276 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8400#issuecomment-134480577
Merged build started.
---
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/8400#issuecomment-134480512
Merged build triggered.
---
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 chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134480016
Yes, that's why I changed the `MAX_SCALE` from `38` to `18`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8400#issuecomment-134482339
@yhuai The Julian day of epoch of unix timestamp (1970-01-01 0:0:0) is
2440587.5, if we use integer for julian days, then it will be cut off at noon
(12pm) of a day. But
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134483133
If you change `MAX_SCALE`, we are changing the maximum scale we can
support, right? I feel the right solution is fixing
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/8391#discussion_r37833992
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -275,6 +275,7 @@ private[sql] class JDBCRDD(
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134487974
```
hive select 10.301 / 1.42040001;
OK
7.253521126760562
```
---
If your project is set up for it, you can
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134487838
I think the maximum scale we can support should be 18, not 38. As you can
see that in `SYSTEM_DEFAULT`, isn't it? I am not sure if that's the original
design,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8405#issuecomment-134454742
[Test build #41507 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41507/console)
for PR 8405 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8405#issuecomment-134454755
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/8405#issuecomment-134454751
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37827836
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37827779
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8400#discussion_r37828796
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala
---
@@ -49,13 +49,18 @@ class DateTimeUtilsSuite extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8388#issuecomment-134464798
[Test build #41503 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41503/console)
for PR 8388 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8388#issuecomment-134464856
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/8388#issuecomment-134464858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8368#discussion_r37832012
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -377,6 +377,60 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8276#issuecomment-134472240
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/8276#issuecomment-134472241
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/8276#issuecomment-134472211
**[Test build #41504 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41504/console)**
for PR 8276 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8394#discussion_r37832964
--- Diff: R/pkg/R/DataFrame.R ---
@@ -945,6 +947,19 @@ setMethod([, signature(x = DataFrame, i =
missing),
select(x, j)
})
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134474759
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/8409#issuecomment-134479215
[Test build #41512 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41512/console)
for PR 8409 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134479241
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/8400#issuecomment-134488218
LGTM now
---
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 zsxwing commented on the pull request:
https://github.com/apache/spark/pull/8405#issuecomment-134489427
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8405#issuecomment-134489312
Merged build started.
---
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/8405#issuecomment-134489297
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8406
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/8406#issuecomment-134457253
I've merged this.
---
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 MechCoder commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37828612
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134462835
[Test build #41511 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41511/console)
for PR 8007 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134462838
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/8276#discussion_r37829318
--- Diff: R/pkg/inst/tests/test_Serde.R ---
@@ -0,0 +1,67 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/8410
[Minor] [Doc] Fix LDA user guide issue
Fix LDA user guide issue, actually we have Python API for LDA.
You can merge this pull request into a Git repository by running:
$ git pull
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8410#issuecomment-134462999
Merged build triggered.
---
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 yanboliang commented on the pull request:
https://github.com/apache/spark/pull/8410#issuecomment-134463030
@mengxr @jkbradley Could we merge it to branch-1.5 ASAP?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134464456
Merged build started.
---
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/8007#issuecomment-134464445
Merged build triggered.
---
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/8391#discussion_r37830657
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -275,6 +275,7 @@ private[sql] class JDBCRDD(
GitHub user sirpkt opened a pull request:
https://github.com/apache/spark/pull/8411
[SPARK-10216][SQL]Avoid creating empty files during overwrite into Hive
table with group by query
I added check routine at saveAsHiveFile() of InsertIntoHiveTable.
It checks whether the given
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8412#issuecomment-134470596
[Test build #41515 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41515/consoleFull)
for PR 8412 at commit
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8233#issuecomment-134471497
Sure, I'll update the code
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/8368#issuecomment-134471517
LGTM. Merging to master and branch-1.5.
---
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/8394#issuecomment-134471688
Merged build triggered.
---
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/8269#issuecomment-134487147
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/8269#issuecomment-134487097
[Test build #41509 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41509/console)
for PR 8269 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8410#issuecomment-134488868
This is fixed in [https://github.com/apache/spark/pull/8254]. Let's just
focus on merging that PR to avoid conflicts. (I'm working on reviewing it but
have to wait
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8406#issuecomment-134455831
[Test build #41496 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41496/console)
for PR 8406 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8406#issuecomment-134455904
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8394#discussion_r37827367
--- Diff: R/pkg/R/DataFrame.R ---
@@ -945,6 +947,19 @@ setMethod([, signature(x = DataFrame, i =
missing),
select(x, j)
})
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8394#discussion_r37827391
--- Diff: R/pkg/inst/tests/test_sparkSQL.R ---
@@ -587,6 +587,29 @@ test_that(select with column, {
expect_equal(collect(select(df3, x))[[1, 1]], x)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8407#issuecomment-134458406
[Test build #41498 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41498/console)
for PR 8407 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8269#issuecomment-134458469
Merged build started.
---
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/8407#issuecomment-134458465
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/8407#issuecomment-134458466
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/8269#issuecomment-134458462
Merged build triggered.
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8400#discussion_r37828840
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala
---
@@ -49,13 +49,18 @@ class DateTimeUtilsSuite extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7884#issuecomment-134461807
Merged build triggered.
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/8400#discussion_r37828834
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala
---
@@ -49,13 +49,18 @@ class DateTimeUtilsSuite extends
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/7884#issuecomment-134461595
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7884#issuecomment-134461819
Merged build started.
---
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/8007#issuecomment-134461805
Merged build triggered.
---
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/8007#issuecomment-134461818
Merged build started.
---
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 dbtsai commented on the pull request:
https://github.com/apache/spark/pull/7884#issuecomment-134461590
Seems Jenkins timeout a lot. Give another shot.
---
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/8408#issuecomment-134466555
[Test build #41501 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41501/console)
for PR 8408 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8408#issuecomment-134466964
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/8408#issuecomment-134466968
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mccheah commented on the pull request:
https://github.com/apache/spark/pull/8007#issuecomment-134468874
Hm, initial testing seems ok. I did run into a case where an asking was
lost. Not immediately clear as to why, but it turned out that the executor had
been removed
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8400#issuecomment-134485906
[Test build #41520 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41520/consoleFull)
for PR 8400 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/8390#issuecomment-134486686
cc @liancheng can you review this? 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 chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/8409#issuecomment-134489197
@davis, are you suggesting we should change the code in
`HiveTypeCoericion.DecimalPrecision` for Div only? not anywhere else?
---
If your project is set up for
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37827592
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37828718
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
GitHub user chenghao-intel opened a pull request:
https://github.com/apache/spark/pull/8409
[SPARK-10215][SQL][WIP]Div of Decimal returns null
```scala
val d = Decimal(1.12321)
val df = Seq((d, 1)).toDF(a, b)
df.selectExpr(b * a / b).collect() // = Array(Row(null))
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7884#issuecomment-134462010
[Test build #41510 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41510/consoleFull)
for PR 7884 at commit
Github user MechCoder commented on a diff in the pull request:
https://github.com/apache/spark/pull/7854#discussion_r37828980
--- Diff: python/pyspark/mllib/linalg/__init__.py ---
@@ -461,32 +461,41 @@ def __init__(self, size, *args):
self.size = int(size)
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/8394#discussion_r37832421
--- Diff: R/pkg/inst/tests/test_sparkSQL.R ---
@@ -587,6 +587,29 @@ test_that(select with column, {
expect_equal(collect(select(df3, x))[[1, 1]],
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/8394#discussion_r37832419
--- Diff: R/pkg/R/DataFrame.R ---
@@ -945,6 +947,19 @@ setMethod([, signature(x = DataFrame, i =
missing),
select(x, j)
601 - 700 of 762 matches
Mail list logo