-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-1537] [YARN] [WiP] Add history provider for YARN Application Timeline Server #5423
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
steveloughran
wants to merge
15
commits into
apache:master
from
steveloughran:stevel/feature/SPARK-1537-ATS
Closed
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
a77505b
SPARK-1537 SPARK-11315 YarnHistoryService patch applied to latest mas…
steveloughran 7812950
SPARK-1537 SPARK-11323 yarn history provider patch applied atop the n…
steveloughran 505975d
SPARK-11315 scaladoc and import cleanup (minor)
steveloughran da311e3
SPARK-11315 event publisher to add a version counter to every entity …
steveloughran 6b078cb
SPARK-11323 yarn provider to use entity version counter to compare an…
steveloughran 9804b9e
SPARK-11315 publish entity version counter as number; no need to stri…
steveloughran 25b1b24
SPARK-11323 tests to get the version counter marshalling working
steveloughran 5aaeb08
SPARK-11323 fixing a regression which turned out to be a typo; lots o…
steveloughran 09df51d
SPARK-11323 scalastyle fix
steveloughran 783fb44
SPARK-11323 chop min refresh interval from 10s to 1s, as testing like…
steveloughran db94b4b
SPARK-11315 ignore BlockUpdated and ExecutorMetricsUpdate events
steveloughran f90bd0b
SPARK-11315 add info-level logging of post to ATS with entity ID and …
steveloughran 03576d4
SPARK-11315 style: methods whose arg list is multiline have newline b…
steveloughran 1c5808a
SPARK-11323 style: methods whose arg list is multiline have newline b…
steveloughran 2a5a739
SPARK-11323 - some more style checking in the rest/ package
steveloughran File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do these render correctly? They all start with
1., unlike the list above.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Markdown takes either; here's [the rendered file|https://github.com/steveloughran/spark/blob/stevel/feature/SPARK-1537-ATS/docs/monitoring.md]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice to know, but it would be good for the source file to be consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll make them all 1.; less maintenance