You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/04/24 00:14:24 UTC
[GitHub] [accumulo-testing] tushardhadiwal opened a new pull request #118: Adding Accumulo Availability Monitor
tushardhadiwal opened a new pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118
Initial code for an accumulo availability monitor which poses as a synthetic user and is doing scans of random values across various tablet servers and capturing timing information related to how long such scans take in order get a overall judgement of availability of all tablet servers.
Monitor.Java : This file includes core code for scan operation that randomly picks a tablet server, identifies its minimum and maximum rows range and picks out 'distance' (configurable through opts) number of rows to be scanned in one iteration.
MonitorOpts.java : This file is used to provide all the parameters which are taken by the Monitor Class. Interesting Options Include : Ability to set scanner sleep time , between iterations. Setting Batch Size that scanner uses internally. Setting number of iteration or continuous mode.
monitor script : This follows same pattern as cingest, rwalk, performance scripts.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] phrocker edited a comment on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
phrocker edited a comment on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620313538
>
>
> This appears to have been merged in [c8ef784](https://github.com/apache/accumulo-testing/commit/c8ef7846b4426c5cb86632ade692b0ff8aa23977)
Yes. I forgot the "This closes..." tag as I did from the command line. I had some mentions from long ago and have run the code to verify. Thanks for putting the commit hash in here @ctubbsii . I've become quite forgetful.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] phrocker commented on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
phrocker commented on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620313538
>
>
> This appears to have been merged in [c8ef784](https://github.com/apache/accumulo-testing/commit/c8ef7846b4426c5cb86632ade692b0ff8aa23977)
Yes. I forgot the "This closes..." tag as I did from the command line. I had some mentions form long ago. Thanks for putting that in here @ctubbsii . I've become quite forgetful.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] ctubbsii commented on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620311768
This appears to have been merged in c8ef7846b4426c5cb86632ade692b0ff8aa23977
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] ctubbsii commented on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620912941
@phrocker You can also just do a squash and merge from the GitHub UI... that also gives you a chance to edit the commit message. This one wasn't well-formatted.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] phrocker commented on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
phrocker commented on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620939217
>
>
> @phrocker You can also just do a squash and merge from the GitHub UI... that also gives you a chance to edit the commit message. This one wasn't well-formatted.
ah yes. I wanted to sign the commit but after pushing I realized that wasn't necessary ( by way of seeing that other commits weren't signed )-- so yeah ... I should have used the UI ( though in reading about it I think that may be possible ). Either way, thanks!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] phrocker edited a comment on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
phrocker edited a comment on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620313538
>
>
> This appears to have been merged in [c8ef784](https://github.com/apache/accumulo-testing/commit/c8ef7846b4426c5cb86632ade692b0ff8aa23977)
Yes. I forgot the "This closes..." tag as I did from the command line. I had some mentions fromlong ago. Thanks for putting the commit hash in here @ctubbsii . I've become quite forgetful.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] phrocker edited a comment on pull request #118: Adding Accumulo Availability Monitor
Posted by GitBox <gi...@apache.org>.
phrocker edited a comment on pull request #118:
URL: https://github.com/apache/accumulo-testing/pull/118#issuecomment-620939217
>
>
> @phrocker You can also just do a squash and merge from the GitHub UI... that also gives you a chance to edit the commit message. This one wasn't well-formatted.
ah yes. I wanted to sign the commit but after pushing ( recently chided on another project for forgetting to do this )-- but I realized that wasn't necessary here after pushing during key verification of this and other commits-- so yeah ... I should have used the UI ( though in reading about it I think that may be possible ). Either way, thanks!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org