Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22602
**[Test build #96852 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96852/testReport)**
for PR 22602 at commit
Github user samdvr commented on the issue:
https://github.com/apache/spark/pull/22596
@dongjoon-hyun merged, thank you, did not know how to skip lint rule there.
---
-
To unsubscribe, e-mail:
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/22609
This is a regression introduced in 2.3; unfortunately we did not notice it
until now.
On Tue, Oct 2, 2018 at 4:56 AM Shahid wrote:
> Hi @mridulm
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22602#discussion_r221934230
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeWriterSuite.scala
---
@@ -0,0 +1,43 @@
+/*
+ *
Github user peter-toth commented on the issue:
https://github.com/apache/spark/pull/22318
@cloud-fan could you please help me with this PR and take it one step
forward?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22606
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22606
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96847/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10989#discussion_r221961271
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastHashJoin.scala
---
@@ -117,6 +120,87 @@ case class BroadcastHashJoin(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22602
**[Test build #96859 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96859/testReport)**
for PR 22602 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22603
**[Test build #96856 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96856/testReport)**
for PR 22603 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22605
**[Test build #96848 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96848/testReport)**
for PR 22605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22607
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22602#discussion_r221955172
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeWriterSuite.scala
---
@@ -0,0 +1,43 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22606
**[Test build #96847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96847/testReport)**
for PR 22606 at commit
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22601
Thanks a lot @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96852/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22596
**[Test build #96858 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96858/testReport)**
for PR 22596 at commit
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
>> So is there no behavior change if your master specifies only one master?
If user specifies only 1 master then there is no behavioral change.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22601
Thank you, @shahidki31 .
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22609
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96846/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22609
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22605
Thanks, @HyukjinKwon !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22609
Hi @mridulm , Can't we limit the task information by setting
'spark.ui.retainedTasks' lesser, to avoid OOM? correct me if I am wrong.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22602
**[Test build #96857 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96857/testReport)**
for PR 22602 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/10989#discussion_r221972056
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastHashJoin.scala
---
@@ -117,6 +120,87 @@ case class BroadcastHashJoin(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22603
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96856/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22603
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96844/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22607
**[Test build #96844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96844/testReport)**
for PR 22607 at commit
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10989#discussion_r221966919
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastHashJoin.scala
---
@@ -117,6 +120,87 @@ case class BroadcastHashJoin(
Github user rezasafi closed the pull request at:
https://github.com/apache/spark/pull/21916
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user rezasafi commented on the issue:
https://github.com/apache/spark/pull/21916
I will close this and open the actual Pr shortly. Thank you everyone for
the great reviews.
---
-
To unsubscribe, e-mail:
Github user rezasafi commented on the issue:
https://github.com/apache/spark/pull/22612
Previously I opened a WIP PR
[here](https://github.com/apache/spark/pull/21916) and
[here](https://github.com/rezasafi/spark/pull/1). I have applied all the
comments that I received there to this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22599
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22599
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96855/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22606
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22596
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96858/
Test PASSed.
---
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22609
@gengliangwang is right; there are still things that need to be done when
the UI is disabled; job, stage and executor info are all still exposed even if
the UI is disabled, and the task even
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22606
@dongjoon-hyun Could you help me fix the website?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22613
cc @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user shahidki31 opened a pull request:
https://github.com/apache/spark/pull/22613
[SPARK-25583][DOC][BRANCH-2.3]Add history-server related configuration in
the documentation.
## What changes were proposed in this pull request?
This is a follow up PR for the PR,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22606
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/22610
Thanks for looking into this @viirya ! I agree that there are lots of
cases where casting to another type is intentional and works fine, so this
isn't a bug. The only other idea I have is to
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22606
Great, @gatorsmile ! Could you update Spark website properly, too?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22605
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96848/
Test PASSed.
---
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22602#discussion_r221960064
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeWriterSuite.scala
---
@@ -0,0 +1,43 @@
+/*
+ *
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22601
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22601
`spark.history.fs.endEventReparseChunkSize` and
`spark.history.fs.inProgressOptimization.enabled` exist since `branch-2.4`. So,
this PR lands at master/2.4. If you want to fix some in
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22613
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22318
For a query similar to the one in the PR description:
`a.join(b, a("id") === b("id"), "inner").join(c, a("id") === b("id"),
"inner"`
It is interpreted as a cartesian products now.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22599
**[Test build #96855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96855/testReport)**
for PR 22599 at commit
Github user rezasafi commented on the issue:
https://github.com/apache/spark/pull/22612
@squito
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22602#discussion_r222032210
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeWriterSuite.scala
---
@@ -0,0 +1,48 @@
+/*
+ *
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22599
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22607
Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22596
BTW, I chose the most official one, `Sam Davarnia `.
But, it seems that your GitHub is not happy with that. Next time, you had
better be consistent in your commit message.
```
Github user samdvr commented on the issue:
https://github.com/apache/spark/pull/22596
Thanks will do!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user samdvr closed the pull request at:
https://github.com/apache/spark/pull/22596
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22524
**[Test build #96860 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96860/testReport)**
for PR 22524 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22596
**[Test build #96858 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96858/testReport)**
for PR 22596 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22596
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22602
**[Test build #96857 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96857/testReport)**
for PR 22602 at commit
Github user peter-toth commented on the issue:
https://github.com/apache/spark/pull/22318
Thanks @viirya, your analysis is correct.
Unfortunately an attribute doesn't have a reference to its dataset so I
don't think this scenario can be solved easily. I believe the good
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22607
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22613
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22613
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/22610#discussion_r222007837
--- Diff: python/pyspark/worker.py ---
@@ -84,13 +84,36 @@ def wrap_scalar_pandas_udf(f, return_type):
arrow_return_type =
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22602
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22596
Since it's merged into `branch-2.2`, could you close this PR? GitHub only
closes the PRs against `master` branch. :)
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22605
Rebased to `master` due to https://github.com/apache/spark/pull/22599.
---
-
To unsubscribe, e-mail:
GitHub user rezasafi opened a pull request:
https://github.com/apache/spark/pull/22612
[SPARK-24958] Add executors' process tree total memory information to
heartbeat signals.
Spark executors' process tree total memory information can be really
useful. Currently such information
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22610#discussion_r222015287
--- Diff: python/pyspark/worker.py ---
@@ -84,13 +84,36 @@ def wrap_scalar_pandas_udf(f, return_type):
arrow_return_type =
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96857/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22602
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/22503
@HyukjinKwon is this ready to be merged in, or is there more feedback to be
addressed?
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22488
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222052506
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceRequestHelper.scala
---
@@ -0,0 +1,92 @@
+/*
+ *
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22491
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96862/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22491
**[Test build #96862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96862/testReport)**
for PR 22491 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22503
**[Test build #96865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96865/testReport)**
for PR 22503 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22500
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22606
You mean http://spark.apache.org/versioning-policy.html and the reference
to 2.4? I think that's still valid. When 2.4 is released, I'd propose to change
that to refer to 3.0 being released .. I
GitHub user kmanamcheri opened a pull request:
https://github.com/apache/spark/pull/22614
HiveClient.getPartitionsByFilter should not throw an exception if HMS
retries directSql
## What changes were proposed in this pull request?
When using partition filter pushdown to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22614
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kmanamcheri commented on the issue:
https://github.com/apache/spark/pull/22614
@mallman @cloud-fan @ericl @rezasafi
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22524
**[Test build #96860 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96860/testReport)**
for PR 22524 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/22609
@vanzin any reason why liveStore is hardcoded to be in-memory ? Any
implications of making it disk backed ? That might be another option to unblock
- requires a config change to existing
Github user michalsenkyr commented on the issue:
https://github.com/apache/spark/pull/22527
I restructured the code in this commit to allow easier addition of
array/list support in the future.
---
-
To unsubscribe,
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22609
Task events still might need to be processed - even for live applications
with the UI disabled. See the API that has been mentioned before
(`SparkStatusTracker`) for why. Processing task events is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22347
**[Test build #96870 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96870/testReport)**
for PR 22347 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96872 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96872/testReport)**
for PR 22047 at commit
1 - 100 of 475 matches
Mail list logo