GitHub user tmagrino opened a pull request:
https://github.com/apache/spark/pull/13861
[SPARK-15885] [Web UI] Provide links to executor logs from stage details
page in UI
## What changes were proposed in this pull request?
This moves over old PR https://github.com/apache/spark/pull/13664 to target
master rather than branch-1.6.
Added links to logs (or an indication that there are no logs) for entries
which list an executor in the stage details page of the UI.
This helps streamline the workflow where a user views a stage details page
and determines that they would like to see the associated executor log for
further examination. Previously, a user would have to cross reference the
executor id listed on the stage details page with the corresponding entry on
the executors tab.
Link to the JIRA: https://issues.apache.org/jira/browse/SPARK-15885
## How was this patch tested?
Ran existing unit tests.
Ran test queries on a platform which did not record executor logs and again
on a platform which did record executor logs and verified that the new table
column was empty and links to the logs (which were verified as linking to the
appropriate files), respectively.
Attached is a screenshot of the UI page with no links, with the new columns
highlighted. Additional screenshot of these columns with the populated links.
Without links:
With links:
This contribution is my original work and I license the work to the project
under the Apache Spark project's open source license.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tmagrino/spark uilogstweak
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/13861.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #13861
----
commit 2f4490840cb796f89f627f1b9105f1658f32e456
Author: Tom Magrino <[email protected]>
Date: 2016-06-10T21:11:30Z
Add links to executor logs from stage details page in UI.
commit 482175294010a01cdd21c9ff60068b0fcc9180a5
Author: Tom Magrino <[email protected]>
Date: 2016-06-10T23:59:08Z
Small fixs for missing indices and testing.
commit 0942220ee1cd7c94c36db2f75670eba44e94d506
Author: Tom Magrino <[email protected]>
Date: 2016-06-13T16:17:59Z
Code style fix.
commit 70d54e801e4b245c06fa111c72e221dedbd08607
Author: Tom Magrino <[email protected]>
Date: 2016-06-14T19:23:24Z
Changes to address UI consistency comments by @ajbozarth.
commit 1a2576d4873a05c9cc4740271224aad81820690d
Author: Tom Magrino <[email protected]>
Date: 2016-06-22T22:15:39Z
Merge branch 'master' into uilogstweak
----
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
