-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-29531 Addendum fix missing charset in RS Status page #7597
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
HBASE-29531 Addendum fix missing charset in RS Status page #7597
Conversation
|
This problem was uncovered in #7568 (comment). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Made sure that the Content-Type HTTP header includes the UTF-8 charset when the RegionServer status web UI page is returned so that it is consistent with Master Status web UI page.
21aae05 to
4a663ac
Compare
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
Some tests failed: TestAcidGuaranteesWith*, TestPrefetchPersistence. I think they are unrelated. |
taklwu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Made sure that the Content-Type HTTP header includes the UTF-8 charset when the RegionServer status web UI page is returned so that it is consistent with Master Status web UI page.