GitHub user kkhatua opened a pull request:

    DRILL-6289: Cluster view should show more relevant information

    The commits make use of AJAX calls by the browser to all the Drillbits for 
status metrics, which are then displayed here. This works with unsecured Drill 
sessions, while secure setups will not be able to show the metrics because of 
certificate authentication.
    To achieve this, the HTTP port needed to be discoverable, for which changes 
to the protobuf were done to provide the HTTP port number as well.
    The Drillbit homepage has been extended to show the following:
    * Heap Memory in use
    * Direct Memory (actively) in use - Since we're not able to get the total 
memory held by Netty at the moment, but only what is currently allocated to 
running queries. The base memory shown is the total peak allocation by Drill 
for Direct, because the actual Direct memory held is not available. In an 
active stable system in production, this should be around 90%.
    * Average (System) Load Factor reported by the OS for that node.
    * Pill indicating the current Drillbit whose WebUI is being viewed 
(existing UX feature), with an option to load (in a new window) the UI from 
other Drillbits.
    Information such as the User, Data and Control port numbers don't help much 
during general cluster health, so it might be worth removing this information 
if more real-estate is needed. For now, we are retaining this. 

You can merge this pull request into a Git repository by running:

    $ git pull DRILL-6289

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1203
commit 94e8bbaefea7c6fe9b5729bcec3cba1dbbb3bc8f
Author: Kunal Khatua <kkhatua@...>
Date:   2018-03-26T05:03:12Z

    Protobuf change to carry HTTP port info

commit 9b1513da949da870fc8d9a96472c4ca307e7d5e8
Author: Kunal Khatua <kkhatua@...>
Date:   2018-04-05T22:21:47Z

    Allow CORS for access to remote Drillbit metrics
    Cross-origin resource sharing (CORS) is required to ensure that the 
WebServer is able serve REST calls for status pages.

commit eb9ae7c36a26194b5a49685230e2a94282efee84
Author: Kunal Khatua <kkhatua@...>
Date:   2018-04-05T21:21:18Z

    Addition of Bootstrap's Glyphicons
    As part of the Bootstrap's components, this meets Apache License criteria
    Remove popout.png

commit 45e4831403997661c1df578a95e3c985f64ff4f8
Author: Kunal Khatua <kkhatua@...>
Date:   2018-04-06T23:08:43Z

    Materialize relevant metrics
    1. Heap memory (incl usage)
    2. Heap memory (incl usage)
    3. Average System Load (last 1 min)
    4. Option to view from other nodes (pop out)
    5. Added Glyphicons

commit dd507ea95a7d0ef4904092ef4fc43ee3b44ef058
Author: Kunal Khatua <kkhatua@...>
Date:   2018-03-30T06:24:54Z

    Update System Table and related tests
    1. Updated System Table to show HTTP port
    2. Updated unit tests



Reply via email to