Commit 7892207
authored
HBASE-29531 Migrate RegionServer Status Jamon page back to JSP (#7371)
This is the 2/3 step of the Jamon to JSP migration: the Region Server Status page.
Did the migration the same way as for the Master Status page: #6875
Migrated the Jamon code to JSP as close as possible. Extracted the duplicated `formatZKString` method to new java class: ZKStringFormatter and added unit tests.
Changed the Region Server Status page back to `/regionserver.jsp`. Made sure that `/rs-status` redirects to `/regionserver.jsp`.
Extracted the BlockCache inline CSS styles to `hbase.css` file. Also extracted the large BlockCache Hit Ratio periods paging JavaScript code to separate .js file.
Introduced a `src/main/resources/hbase-webapps/common` directory where we can place common JSP files which are used by both Master and RegionServer JSP pages. This required to adjust the JSP compiler Maven Antrun plugin a bit.
Extracted the inline tablesorter initialization JavaScript code to separate file.
Signed-off-by: Duo Zhang <zhangduo@apache.org>1 parent fafa03c commit 7892207
File tree
59 files changed
+2547
-2068
lines changed- hbase-server
- src
- main
- jamon/org/apache/hadoop/hbase/tmpl
- common
- regionserver
- java/org/apache/hadoop/hbase
- master/http
- regionserver/http
- util
- resources/hbase-webapps
- common
- master
- regionserver
- static
- css
- js
- test/java/org/apache/hadoop/hbase
- master/http
- regionserver/http
- util
- hbase-shell/src/main/ruby/hbase
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
59 files changed
+2547
-2068
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
564 | 564 | | |
565 | 565 | | |
566 | 566 | | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
567 | 581 | | |
568 | 582 | | |
569 | 583 | | |
570 | | - | |
| 584 | + | |
571 | 585 | | |
572 | | - | |
| 586 | + | |
573 | 587 | | |
574 | 588 | | |
575 | 589 | | |
| |||
Lines changed: 0 additions & 140 deletions
This file was deleted.
0 commit comments