kevinrr888 commented on PR #4524: URL: https://github.com/apache/accumulo/pull/4524#issuecomment-2361698043
@kevinrr888 - > This was an existing TODO that you had marked in your "distributed FATE" WIP PR: > // TODO 4131 > // TODO make the max time a function of the number of concurrent callers, as the number of > // concurrent callers increases then increase the max wait time > // TODO could support signaling within this instance for known events > // TODO made the maxWait low so this would be responsive... that may put a lot of load in > // the case there are lots of things waiting... > I made the max wait = curr num callers in seconds. Does this address the first TODO? For the other two TODOs, I was not sure what was to be done, so I left them. Can these be safely deleted without creating follow on issue(s)? If follow on issues should be made, perhaps it would be best for you to write these or explain it here because I am not sure what should be done. @keith-turner - > Yeah you can delete those. I will look into it and open an issue if needed. This may still need to be looked into just posting this as a reminder just in case -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
