You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ddanielr (via GitHub)" <gi...@apache.org> on 2024/05/23 11:15:17 UTC
[PR] Removes Error and drops logging level on message [accumulo]
ddanielr opened a new pull request, #4593:
URL: https://github.com/apache/accumulo/pull/4593
This log message is encountered as the TServer Util attempts port binding.
This is a log message that is normally occuring when running more than a single tserver process per node. Removed the word "Error" since that creates additional background noise when searching logs.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Removes Error and drops logging level on message [accumulo]
Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr commented on PR #4593:
URL: https://github.com/apache/accumulo/pull/4593#issuecomment-2135921286
Added the logic to modify the message if port search was enabled.
Tested this with enabling and disabling the `tserver.port.search` property and running multiple tservers locally.
`tserver.port.search=true`
```
[rpc.TServerUtils] DEBUG: Failed attempting to create server at localhost:10000. Could not create ServerSocket on address localhost/127.0.0.1:10000.
```
`tserver.port.search=false`
```
[rpc.TServerUtils] WARN : Error attempting to create server at localhost:9997. Error: Could not create ServerSocket on address localhost/127.0.0.1:9997.
```
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Removes Error and drops logging level on message [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #4593:
URL: https://github.com/apache/accumulo/pull/4593#issuecomment-2126965630
Given that it's expected that this condition may occur when running multiple servers per host, then we might want to only lower the log level when the port search property is true. It looks like there is a clear path for passing a boolean down to this method to reflect whether the method is being called from some code that is, or is not, handling the port search 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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Removes Error and drops logging level on message [accumulo]
Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr commented on PR #4593:
URL: https://github.com/apache/accumulo/pull/4593#issuecomment-2127198232
> Given that it's expected that this condition may occur when running multiple servers per host, then we might want to only lower the log level when the port search property is true. It looks like there is a clear path for passing a boolean down to this method to reflect whether the method is being called from some code that is, or is not, handling the port search case.
That's a good idea. I'll work on adding that to this PR.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Removes Error and drops logging level on message [accumulo]
Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr merged PR #4593:
URL: https://github.com/apache/accumulo/pull/4593
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org