GitHub user zentol opened a pull request:

    https://github.com/apache/flink/pull/3376

    [FLINK-5870] Handlers define REST URLs

    This PR allows ```RequestHandler```s to define the REST URLs under which 
they should be registered.
    
    For this purpose the following method was added to the ```RequestHandler``` 
 interface: ```String[] getPaths();```
    
    Additionally, a utility class ```RestUtils``` was added that contains a 
number of often used REST URL components as constants (things like "job" or 
":jobid"), and a utility method to concatenate these components into a REST URL 
(basically concat with "/"). The idea here is to prevent typos and such.
    
    Tests were added for every single handler to verify that the correct paths 
are returned. As a result if any URL should be changed, which isn't allowed 
since the REST API is considered stable, a test will now fail.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zentol/flink 5870_handler_url

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3376.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 #3376
    
----
commit 6f0987f46afc267023b9834a3d7abd4e5762f2e1
Author: zentol <[email protected]>
Date:   2017-02-20T15:29:27Z

    [FLINK-5870] Handlers define REST URLs

----


---
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.
---

Reply via email to