GitHub user srishtyagrawal opened a pull request:

    https://github.com/apache/storm/pull/2505

    STORM-2877: Add an option to configure pagination in Storm UI

    The current pagination default value for Storm UI is hard-coded to be 20. 
Pagination has been introduced in Storm 1.x. Having 20 items in the list 
restricts searching through the page. It will be beneficial to have a 
configuration option, `ui.pagination`, which will set the pagination value when 
Storm UI loads. This option can be added to storm.yaml along with other 
configurations.
    
    Changed the value of `ui.pagination` in `storm.yaml` to test the following 
cases :
    
    
    
    | Value in `storm.yaml`        | Result    
    | ------------- |:-------------:| 
    |  No `ui.pagination`     | defaults to 20 entries |
    |  `“All”`     | Throws error, expects integer |
    |  `-10`     | List doesn't have any items in it |
    |  `30`     | Populates list with 30 entries as default |
    |  `10`     | Populates list with 10 entries as default|
    |  `10`     | Populates list with 10 entries as default|
    |  `-1`      | Equivalent of All from the drop-down options|

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

    $ git pull https://github.com/srishtyagrawal/storm STORM-2877

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

    https://github.com/apache/storm/pull/2505.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 #2505
    
----
commit 3b1bfdbdbe98a141e20f475c8bc08d562a7bda0a
Author: Srishty Agrawal <sagrawal@...>
Date:   2018-01-03T22:51:25Z

    STORM-2877: Add an option to configure pagination in Storm UI

----


---

Reply via email to