Skip to content
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

STORM-2877: Backport for 1.0.x-branch #2535

Merged
merged 1 commit into from Feb 1, 2018

Conversation

@srishtyagrawal
Copy link

srishtyagrawal commented Jan 25, 2018

Backporting this Storm UI change which has already been merged in master.

JIRA ticket: https://issues.apache.org/jira/browse/STORM-2877
Merged PR in master branch: #2505

@srishtyagrawal srishtyagrawal force-pushed the srishtyagrawal:STORM-2877-1.0.x branch 2 times, most recently from abd13d7 to 0612470 Jan 25, 2018
@srishtyagrawal srishtyagrawal force-pushed the srishtyagrawal:STORM-2877-1.0.x branch from 0612470 to 70e1112 Jan 25, 2018
@srishtyagrawal srishtyagrawal changed the title STORM-2877: Add an option to configure pagination in Storm UI STORM-2877: Backport for Storm-1.0.x-branch Jan 25, 2018
@srishtyagrawal srishtyagrawal changed the title STORM-2877: Backport for Storm-1.0.x-branch STORM-2877: Backport for 1.0.x-branch Jan 25, 2018
Copy link
Contributor

HeartSaVioR left a comment

+1

@HeartSaVioR

This comment has been minimized.

Copy link
Contributor

HeartSaVioR commented Feb 1, 2018

@srishtyagrawal
Could you also raise pull request against 1.1.x branch?

@asfgit asfgit merged commit 70e1112 into apache:1.0.x-branch Feb 1, 2018
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
@srishtyagrawal

This comment has been minimized.

Copy link
Author

srishtyagrawal commented Feb 1, 2018

Definitely @HeartSaVioR! Thanks for merging this one.

@srishtyagrawal

This comment has been minimized.

Copy link
Author

srishtyagrawal commented Feb 1, 2018

@HeartSaVioR I see STORM-2877 in the commit history for 1.1.x branch. Did you cherry-pick the changes already? I don't need to create a backport PR now, right?

@HeartSaVioR

This comment has been minimized.

Copy link
Contributor

HeartSaVioR commented Feb 1, 2018

@srishtyagrawal Yes right, the cherry-picking was hopefully easy one. :)

@srishtyagrawal

This comment has been minimized.

Copy link
Author

srishtyagrawal commented Feb 1, 2018

Thanks for confirming that @HeartSaVioR. We have the changes forSTROM-2877-Introduce an option to configure pagination in Storm UI in all the development branches which are 1+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.