Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20216
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20216
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/20216
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
Please help merge code, thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20216
**[Test build #4056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4056/testReport)**
for PR 20216 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20216
**[Test build #4056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4056/testReport)**
for PR 20216 at commit
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
Yes, it makes the Workers / Apps lists collapsible in the same way as other
blocks.
---
-
To unsubscribe, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20216
This basically just makes the Workers / Apps lists collapsible in the same
way as other blocks? that sounds fine.
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20216
**[Test build #4055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4055/testReport)**
for PR 20216 at commit
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20216
LGTM now
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
I agree with your second suggestion, before I did not understand what you
mean, now I passed the test I understand what you mean.
1.In order for collapsible tables to persist on
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20216
Thanks for updating the first one, but I stand by the second. The UI should
be consistent and in the previous use of this feature it persisted so it should
here, even if it's not necessary every
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
@ajbozarth
The first suggestion, I have already fixed it.
---
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
1. The first suggestion, I will fix it.
2. The second suggestion, I think it is not necessary. Because spark system
is small, such as 3 workers, do not need to hide the table from the
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
@ajbozarth @srowen
Fix the code, increase the arrow of the form page, maintain the consistency
of the function.
after fix:
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
---
-
To
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20216
Hiding Tables like this has been done elsewhere in the UI (can't remember
where right now though). In that case there's a triangle marker next to the
Header that shows that it's collapsable and
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
No, just hide the table, in fact, the data is already on the page, but we
can not see.
When we refresh the page, it will re-show all the data.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20216
But @guoxiaolongzte your change doesn't cause it to load less data, right?
---
-
To unsubscribe, e-mail:
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/20216
Dear Sir, However, the real spark big data environment, a very large number
of workers, every day running a very large number of applications, has
completed a very large number of
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20216
Since this is a UI change can you post screenshots, and I'm not 100%
convinced on the necessity of this when we already have jump links at the top
of most pages.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20216
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
23 matches
Mail list logo
