Github user joshelser commented on the issue:
https://github.com/apache/accumulo/pull/242
Dumping a couple of things here that I'm finding (they're critical comments
-- but they're not to be taken as insults)
* I think there should be an "Apache Accumulo" in the top-left of the
header (instead of the instance name). Perhaps center the instance name
instead? Right now we have 0 branding of our product. We should do better here
(Name, Accumulo logo, ASF logo, etc). Could easily put something into the
footer.
* When the Monitor is refreshing the graphs, the UI seems to be blocked
with auto-refresh -- you can't click any of the drop-downs. This is very
painful as, if you're not watching the ajax calls happening, sometimes your
button clicks are just lost.
* Auto-refresh also appears to make 13 ajax calls per refresh which is
pretty excessive at a refresh of 5s. We have the logic broken up which is nice,
but when we're refreshing all of these, we should minimize the number of ajax
calls we're making (get all the data in one call)
* On the tables view, why do "default" and "accumulo" get auto-added when I
select "all"? This seems repetitive.
* We pull a lot of js files and load them. IIRC, we should try to get down
to one JS file for Accumulo (maybe there's a maven plugin we could use to keep
the separation in source but concatenate them at build time so we only include
one in the HTML).
* OS load should be truncated to a few digits (tenth or hundreth place).
* Is it possible to create some docs about the REST API and publish those
with the monitor? We have this great API but no information about the calls and
the data. The XML and JSON stuff is there for old users, but we should try to
push the new people to the new REST calls.
* Tool-tips are a little intrusive. If I want to sort on a new column, they
pop up really fast and make me move my cursor to view the top elements. Maybe
make them slower to appear or semi-transparent?
* Not all table column headers have tool-tips. We should make a pass for
clarity/simplicity.
* ~~Accumulo instance ID is shown nowhere that I've found. Nor Accumulo
version (info lost over the old monitor).~~ **Found it** in the "info" menu. My
gut reaction is that we could put instanceid and version somewhere more
prominently (it's a common thing I check when debugging user systems), but I
think we could drop the date (we're not a clock, we're a database!)
Looks OK on my phone too (chrome), as well as FF and Chrome on my mac --
woo. I don't have IE at work. I'll try to remember to check when I get home..
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
