[GitHub] accumulo pull request #281: ACCUMULO-4655 Added a Response Time column to th...

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo pull request #281: ACCUMULO-4655 Added a Response Time column to th...

ctubbsii
GitHub user ivakegg opened a pull request:

    https://github.com/apache/accumulo/pull/281

    ACCUMULO-4655 Added a Response Time column to the monitor

    Added a Response Time column to the monitor which shows how long it took to get status from that tserver.
    This is useful in determining whether long Last Contact times are dues to one or more tservers or whether
    this is due to churn in the master.

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

    $ git pull https://github.com/ivakegg/accumulo ACCUMULO-4655-2.0

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

    https://github.com/apache/accumulo/pull/281.patch

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

    This closes #281
   
----
commit ebc7dd6921f82e38ebbeb04917cd11c2ae28dc1b
Author: Ivan Bella <[hidden email]>
Date:   2017-07-21T14:19:07Z

    ACCUMULO-4655 Added a Response Time column to the monitor
   
    Added a Response Time column to the monitor which shows how long it took to get status from that tserver.
    This is useful in determining whether long Last Contact times are dues to one or more tservers or whether
    this is due to churn in the master.

----


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #281: ACCUMULO-4655 Added a Response Time column to the monit...

ctubbsii
Github user ivakegg commented on the issue:

    https://github.com/apache/accumulo/pull/281
 
    This is the merge of the ACCUMULO-4655 from 1.8.2 into master.  I updated the tablets.js and the TabletServerInformation to include this in the new monitor tservers page.


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #281: ACCUMULO-4655 Added a Response Time column to the monit...

ctubbsii
In reply to this post by ctubbsii
Github user lstav commented on the issue:

    https://github.com/apache/accumulo/pull/281
 
    Did you branched out of the work I did on #280? I noticed that your code has the version variable introduced on that PR.


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #281: ACCUMULO-4655 Added a Response Time column to the monit...

ctubbsii
In reply to this post by ctubbsii
Github user ivakegg commented on the issue:

    https://github.com/apache/accumulo/pull/281
 
    I branched off of master.  As you noted it appears that #280 was commited into master but was not closed._


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo issue #281: ACCUMULO-4655 Added a Response Time column to the monit...

ctubbsii
In reply to this post by ctubbsii
Github user lstav commented on the issue:

    https://github.com/apache/accumulo/pull/281
 
    Thanks, I'll close #280 to avoid future confusions.


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] accumulo pull request #281: ACCUMULO-4655 Added a Response Time column to th...

ctubbsii
In reply to this post by ctubbsii
Github user asfgit closed the pull request at:

    https://github.com/apache/accumulo/pull/281


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Loading...