You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ctubbsii (via GitHub)" <gi...@apache.org> on 2024/12/11 21:12:50 UTC
[PR] Fix accumulo-cluster [accumulo]
ctubbsii opened a new pull request, #5167:
URL: https://github.com/apache/accumulo/pull/5167
* Update help descriptions for deprecated items
* Streamline debug output and add "DEBUG:" prefix
* Bash grammar fixes (double square brackets, remove unneeded quotes/curlies, etc.)
* Remove eval (use `bash -c` for local case instead)
* Use printf instead of bash 4.4 features for printing command to execute
* Use local variables where possible
* Remove unneded quoting in calls to accumulo-service using guilty knowledge of the fact that most of the options passed will be scalars without any spaces or special characters (improves readability)
* Fix broken computation of localhost addresses by filtering getent entries
--
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] Fix accumulo-cluster [accumulo]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #5167:
URL: https://github.com/apache/accumulo/pull/5167#issuecomment-2540134960
Backport to 2.1 available in #5174
--
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] Fix accumulo-cluster [accumulo]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #5167:
URL: https://github.com/apache/accumulo/pull/5167#issuecomment-2539410566
I'm working on backporting these fixes to 2.1 and 3.1, along with the other recent changes to the scripts, so that they are more similar across branches, and easier to maintain.
--
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] Fix accumulo-cluster [accumulo]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii merged PR #5167:
URL: https://github.com/apache/accumulo/pull/5167
--
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