Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22668
@felixcheung
Thanks for merging.
My account name in jira is : shivuson...@gmail.com
This issue is assigned to me
Github user shivusondur commented on a diff in the pull request:
https://github.com/apache/spark/pull/22677#discussion_r226439337
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/AsyncEventQueue.scala ---
@@ -169,7 +169,8 @@ private class AsyncEventQueue
Github user shivusondur commented on a diff in the pull request:
https://github.com/apache/spark/pull/22677#discussion_r226191226
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/AsyncEventQueue.scala ---
@@ -169,7 +169,8 @@ private class AsyncEventQueue
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22677
cc @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22677
Hi,
I have modified the title and description according to code change. Build
also passed.
Kindly check and merge
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22677
@jiangxb1987 @dongjoon-hyun
I have modified accordingly.
Tested locally with existing UT (to make drop the event, locally
"LISTENER_BUS_EVENT_QUEUE_CAPACITY" is set to
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22668
@felixcheung
OK. Thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22668
@gengliangwang @felixcheung If everything okay, can you please merge the
PR.
---
-
To unsubscribe, e-mail: reviews
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22677
@jiangxb1987
Thanks for your comment,
i think printing "since Wed Dec 31 16:00:00 PST 1969" still looks strange,
Instead we can print "**since start of the applicat
Github user shivusondur commented on a diff in the pull request:
https://github.com/apache/spark/pull/22668#discussion_r224337547
--- Diff: core/src/main/scala/org/apache/spark/ui/PagedTable.scala ---
@@ -154,9 +150,6 @@ private[ui] trait PagedTable[T
Github user shivusondur commented on a diff in the pull request:
https://github.com/apache/spark/pull/22668#discussion_r224318421
--- Diff: core/src/main/scala/org/apache/spark/ui/PagedTable.scala ---
@@ -123,10 +123,9 @@ private[ui] trait PagedTable[T
Github user shivusondur commented on a diff in the pull request:
https://github.com/apache/spark/pull/22677#discussion_r224007729
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/AsyncEventQueue.scala ---
@@ -159,6 +159,11 @@ private class AsyncEventQueue
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22677
cc @HyukjinKwon , @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
GitHub user shivusondur opened a pull request:
https://github.com/apache/spark/pull/22677
[SPARK-25683] Make AsyncEventQueue.lastReportTimestamp inital value as the
currentTime instead of 0
## What changes were proposed in this pull request?
In AsyncEventQueue
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22567
@vanzin
I modified according to your suggestion and updated the code.
Now History properties will come as seperate tab in the EnvironmentPage as
shown below snap.
![screenshot
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22567
@vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22668
@felixcheung If the user inputs **rowcount** in **Show** tab such that all
the rows comes in one page,
From here on user can't see the pagination and he will not able to inputs
**rowcount
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22669
@felixcheung I searched in all the code base, only this place i found
---
-
To unsubscribe, e-mail: reviews-unsubscr
GitHub user shivusondur opened a pull request:
https://github.com/apache/spark/pull/22669
[SPARK-25677] [Spark Compression] spark.io.compression.codec =
org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException
Exception
## What changes were proposed in this pull
GitHub user shivusondur opened a pull request:
https://github.com/apache/spark/pull/22668
[SPARK-25675] [Spark Job History] Job UI page does not show pagination with
one page
## What changes were proposed in this pull request?
Currently in PagedTable.scala pageNavigation
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22567
@vanzin Please review this patch.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user shivusondur reopened a pull request:
https://github.com/apache/spark/pull/22567
[WIP]Environment Page of Spark Job History UI showing wrong value for
spark.ui.retainedJobs
## What changes were proposed in this pull request?
Currently, in History page, Number of Jobs
Github user shivusondur closed the pull request at:
https://github.com/apache/spark/pull/22567
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
GitHub user shivusondur opened a pull request:
https://github.com/apache/spark/pull/22567
[SPARK-25550] [Spark Job History] Environment Page of Spark Job History UI
showing wrong value for spark.ui.retainedJobs
## What changes were proposed in this pull request?
In history page
24 matches
Mail list logo