Skip to content

HOTFIX: fix even/odd rows for upgrade.from config - #7837

Merged
bbejeck merged 1 commit into
apache:trunkfrom
ableegoldman:HOTFIX-fix-html-rows-for-2.4-config
Dec 23, 2019
Merged

HOTFIX: fix even/odd rows for upgrade.from config#7837
bbejeck merged 1 commit into
apache:trunkfrom
ableegoldman:HOTFIX-fix-html-rows-for-2.4-config

Conversation

@ableegoldman

Copy link
Copy Markdown
Member

Minor followup to #7825, should be cherry-picked to 2.4

@bbejeck bbejeck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @ableegoldman did you render this locally to confirm?

@ableegoldman

Copy link
Copy Markdown
Member Author

No...but I noticed in another PR that several other configs also break this even/odd pattern, yet as far as we could tell there are no visible effects.
(That said, it does seem good to avoid breaking it further -- and note that we actually need to cherry-pick this PR to 2.4 first, then apply this fix on top)

@bbejeck

bbejeck commented Dec 23, 2019

Copy link
Copy Markdown
Member

Merged #7837 into trunk.

bbejeck pushed a commit that referenced this pull request Dec 23, 2019
Minor followup to #7825, should be cherry-picked to 2.4

Reviewers: Bill Bejeck <bbejeck@gmail.com>
@bbejeck

bbejeck commented Dec 23, 2019

Copy link
Copy Markdown
Member

cherry-picked to 2.4

@ableegoldman
ableegoldman deleted the HOTFIX-fix-html-rows-for-2.4-config branch December 31, 2019 00:26
qq619618919 pushed a commit to qq619618919/kafka that referenced this pull request May 12, 2020
Minor followup to apache#7825, should be cherry-picked to 2.4

Reviewers: Bill Bejeck <bbejeck@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants