Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14552
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63426/
Test FAILed.
---
GitHub user lshmouse opened a pull request:
https://github.com/apache/spark/pull/14561
SPARK-16972: Move DriverEndpoint out of CoarseGrainedSchedulerBackend
## What changes were proposed in this pull request?
Move DriverEndpoint out of CoarseGrainedSchedulerBackend and make the t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14557
**[Test build #63434 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63434/consoleFull)**
for PR 14557 at commit
[`9ea08e8`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14561
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 feat
Github user GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74026903
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14561
Jenkins 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14561
**[Test build #63435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63435/consoleFull)**
for PR 14561 at commit
[`def6954`](https://github.com/apache/spark/commit/d
Github user GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74027475
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14561
**[Test build #63435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63435/consoleFull)**
for PR 14561 at commit
[`def6954`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14561
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14561
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63435/
Test FAILed.
---
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74027679
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14534
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63433/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14534
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14534
**[Test build #63433 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63433/consoleFull)**
for PR 14534 at commit
[`0a436a0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63427 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63427/consoleFull)**
for PR 14559 at commit
[`b302b1c`](https://github.com/apache/spark/commit/
Github user GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74028584
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63427/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
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
e
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74028772
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14547
**[Test build #63428 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63428/consoleFull)**
for PR 14547 at commit
[`b4e5e6c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14547
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
e
Github user SaintBacchus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14534#discussion_r74028969
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/server/SparkSQLOperationManager.scala
---
@@ -39,15 +38,19 @@ pri
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14547
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63428/
Test FAILed.
---
Github user GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74029250
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -447,7 +447,16 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14534#discussion_r74029654
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/server/SparkSQLOperationManager.scala
---
@@ -39,15 +38,19 @@ private[t
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14414
Merged to master
---
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 i
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14414
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14477
@srowen Sorry for the late reply, this 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 feat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63436 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63436/consoleFull)**
for PR 14559 at commit
[`6a3cb42`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63436/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63436/consoleFull)**
for PR 14559 at commit
[`6a3cb42`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63437/consoleFull)**
for PR 14559 at commit
[`eb0656b`](https://github.com/apache/spark/commit/e
Github user lshmouse commented on the issue:
https://github.com/apache/spark/pull/14561
Jenkins 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 wishe
Github user lshmouse commented on the issue:
https://github.com/apache/spark/pull/14561
Jenkins 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 wishe
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14562
cc @yhuai @liancheng @clockfly
---
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 a
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/14562
[SPARK-16973][SQL] remove the buffer offsets in ImperativeAggregate
## What changes were proposed in this pull request?
the `mutableAggBufferOffset` and `inputAggBufferOffset` in
`Impera
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14562
**[Test build #63438 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63438/consoleFull)**
for PR 14562 at commit
[`9658056`](https://github.com/apache/spark/commit/9
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14554
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14554
LGTM, merging to master!
this conflicts with 2.0, so I didn't backport
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14102#discussion_r74034073
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -35,184 +34,337 @@ import org.apache.spark.ut
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14525
**[Test build #63429 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63429/consoleFull)**
for PR 14525 at commit
[`a48be1f`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14102#discussion_r74034675
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -35,184 +34,337 @@ import org.apache.spark.ut
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/14519
I tested this locally and verified the serialization reduction. I posted
the shuffle size comparison diagram in the PR description. I will merge this
into master. Thanks for your review! @dbtsai
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14525
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63429/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14525
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
e
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14519
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
GitHub user ash211 opened a pull request:
https://github.com/apache/spark/pull/14563
Typo: Fow -> For
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ash211/spark patch-8
Alternatively you can review and apply these changes as t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13868
**[Test build #63430 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63430/consoleFull)**
for PR 13868 at commit
[`ea24b59`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13868
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13868
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63430/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14563
**[Test build #63439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63439/consoleFull)**
for PR 14563 at commit
[`c572942`](https://github.com/apache/spark/commit/c
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14563
Alright, pretty trivial but a typo nonetheless. If you have time to
spell-check at least the same file, we might batch together more changes and
avoid piecemeal review of tiny PRs
---
If your proje
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14560
**[Test build #63431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63431/consoleFull)**
for PR 14560 at commit
[`b8a5267`](https://github.com/apache/spark/commit/
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/14564
[SPARK-16886][EXAMPLES][DOCS] Fix some examples to be consistent and
woring, and indentation in documentation
## What changes were proposed in this pull request?
Originally this PR was
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14564
Hi @srowen , Could you please take a look?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14560
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63431/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14560
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
e
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14564#discussion_r74036579
--- Diff:
examples/src/main/java/org/apache/spark/examples/sql/streaming/JavaStructuredNetworkWordCountWindowed.java
---
@@ -75,28 +75,30 @@ public stat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14564
**[Test build #63440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63440/consoleFull)**
for PR 14564 at commit
[`3615806`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14560
**[Test build #63432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63432/consoleFull)**
for PR 14560 at commit
[`d11ce1f`](https://github.com/apache/spark/commit/
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14564
OK, seems reasonable.
---
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,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14560
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63432/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14560
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
e
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13680#discussion_r74038625
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/UnsafeArraySuite.scala
---
@@ -18,27 +18,131 @@
package org.apache.spark.sql.cat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14537
**[Test build #63441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63441/consoleFull)**
for PR 14537 at commit
[`4ae92d8`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14564
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14564
**[Test build #63440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63440/consoleFull)**
for PR 14564 at commit
[`3615806`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14564
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63440/
Test PASSed.
---
GitHub user uzadude opened a pull request:
https://github.com/apache/spark/pull/14565
KCore algorithm
## What changes were proposed in this pull request?
Added [KCore]
(https://en.wikipedia.org/wiki/Degeneracy_(graph_theory)#k-Cores) algorithm
implementation
Github user uzadude commented on the issue:
https://github.com/apache/spark/pull/14565
I also opened a Jira feature request [[SPARK-16976]]
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have thi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14557
**[Test build #63434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63434/consoleFull)**
for PR 14557 at commit
[`9ea08e8`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14557
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63434/
Test PASSed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14102#discussion_r74041085
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -35,184 +34,337 @@ import org.apache.spark.
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14102#discussion_r74041126
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -35,184 +34,337 @@ import org.apache.spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14557
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
e
Github user sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
ok, will submit another PR for 2.0 branch
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14565
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 feat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14562
**[Test build #63438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63438/consoleFull)**
for PR 14562 at commit
[`9658056`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14562
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63438/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14562
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63437 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63437/consoleFull)**
for PR 14559 at commit
[`eb0656b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63437/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
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
e
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14102#discussion_r74047137
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -35,184 +34,337 @@ import org.apache.spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14563
**[Test build #63439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63439/consoleFull)**
for PR 14563 at commit
[`c572942`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14563
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14563
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63439/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14537
**[Test build #63441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63441/consoleFull)**
for PR 14537 at commit
[`4ae92d8`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14537
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14537
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63441/
Test PASSed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14565
Hi @uzadude, actually, there is a contribution guide here,
https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark. It
would be even nicer if this contribution follows the guidel
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14562
Does this have performance implications? We are adding a layer of
indirection to a hot code path.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14562#discussion_r74051693
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/slicedRows.scala
---
@@ -0,0 +1,108 @@
+/*
+ * Licensed to the Ap
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14562#discussion_r74052366
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -219,111 +219,118 @@ sealed abstract cl
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14562
@hvanhovell I'm not sure about the performance, will benchmark it later,
hopefully they can be inlined by JVM successfully.
---
If your project is set up for it, you can reply to this email and h
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14562#discussion_r74055718
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -219,111 +219,118 @@ sealed abstract cla
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/12990
Sorry I haven't had time to look at this in great detail but have some
concerns.
So the way I read this, it will only ever show 1000 tasks on the task table
(by default at least)? I don'
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14539#discussion_r74062505
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1563,17 +1563,21 @@ object DecimalAggregates extend
101 - 200 of 768 matches
Mail list logo