accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From joshelser <...@git.apache.org>
Subject [GitHub] accumulo issue #242: ACCUMULO-2181/3005 REST API and new Monitor UI
Date Thu, 13 Apr 2017 23:12:33 GMT
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message