GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/19748
[SPARK-20649][core] Simplify REST API resource structure.
With the new UI store, the API resource classes have a lot less code,
since there's no need for complicated translations between the UI
types and the API types. So the code ended up with a bunch of files
with a single method declared in them.
This change re-structures the API code so that it uses less classes;
mainly, most sub-resources were removed, and the code to deal with
single-attempt and multi-attempt apps was simplified.
The only change was the addition of a method to return a single
attempt's information; that was missing in the old API, so trying
to retrieve "/v1/applications/appId/attemptId" would result in a
404 even if the attempt existed (and URIs under that one would
return valid data).
The streaming API resources also overtook the same treatment, even
though the data is not stored in the new UI store.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vanzin/spark SPARK-20649
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19748.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 #19748
----
commit fec91b6d9d7728f9fdfe96fbacf34d8f4d387a83
Author: Marcelo Vanzin <[email protected]>
Date: 2017-03-23T18:10:40Z
[SPARK-20649][core] Simplify REST API resource structure.
With the new UI store, the API resource classes have a lot less code,
since there's no need for complicated translations between the UI
types and the API types. So the code ended up with a bunch of files
with a single method declared in them.
This change re-structures the API code so that it uses less classes;
mainly, most sub-resources were removed, and the code to deal with
single-attempt and multi-attempt apps was simplified.
The only change was the addition of a method to return a single
attempt's information; that was missing in the old API, so trying
to retrieve "/v1/applications/appId/attemptId" would result in a
404 even if the attempt existed (and URIs under that one would
return valid data).
The streaming API resources also overtook the same treatment, even
though the data is not stored in the new UI store.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
