Github user steveloughran commented on the pull request:
https://github.com/apache/spark/pull/5423#issuecomment-117066169
OK, I've done the shallow changes, will now
1. rename health check methods to make clear what they are doing is more
connectivity check
2. add the attempt support & tests
Once I've done that and the tests are passing I'll split up the patch for
review âthough everyone has to recognise that the two patches need to be
fully consistent to work.
w.r.t the moving target comment, there are other changes in the history
code which I've had to address, (e.g REST API), while I've left things alone
to the core that also need to be fixed (message when there's no data should be
provider specific, caching of incomplete apps should be time-bounded, etc,
etc). It is changing, and as with any long-lived branch, it takes effort to
sync up when you are not in the core codebase.
---
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]
