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

2017-06-23 Thread milleruntime
Github user milleruntime commented on the issue: https://github.com/apache/accumulo/pull/271 @ivakegg Are you planning on merging this forward to master? We have yet to merge the new Monitor so I am wondering what would be the best technique to make sure this doesn't get lost. Maybe

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

2017-06-26 Thread ivakegg
Github user ivakegg commented on the issue: https://github.com/apache/accumulo/pull/271 @milleruntime Yes, I would like this feature to persist. I expect this would be merged into the master when complete. If that is before the Monitor PR gets merge, then the person to merge the Mon

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

2017-06-26 Thread milleruntime
Github user milleruntime commented on the issue: https://github.com/apache/accumulo/pull/271 Ok sounds good. I was just thinking ahead since TServersServlet.java is removed in that PR. It shouldn't be a problem finding a spot for your new stuff though... it should actually be easier

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

2017-07-18 Thread ivakegg
Github user ivakegg commented on the issue: https://github.com/apache/accumulo/pull/271 OK, I am going to attempt to merge this in tomorrow unless there are any more comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we