You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by "DomGarguilo (via GitHub)" <gi...@apache.org> on 2026/08/12 16:52:40 UTC
[PR] Support Accumulo 4 in uno ashell [accumulo-uno]
DomGarguilo opened a new pull request, #316:
URL: https://github.com/apache/accumulo-uno/pull/316
Adds a branch to properly handle Accumulo 4's behavior which drops the -u and -p shell connection options and instead use the existing accumulo-client.properties file. For accumulo 2.x and 3.x, continue passing the -u and -p
Verified things work on accumulo 2.1.5 and 4.0
--
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: commits-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support Accumulo 4 in uno ashell [accumulo-uno]
Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo merged PR #316:
URL: https://github.com/apache/accumulo-uno/pull/316
--
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] Support Accumulo 4 in uno ashell [accumulo-uno]
Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on code in PR #316:
URL: https://github.com/apache/accumulo-uno/pull/316#discussion_r3770008554
##########
bin/impl/commands.sh:
##########
@@ -287,7 +287,11 @@ function uno_status_main() {
function uno_ashell_main() {
check_dirs ACCUMULO_HOME || return 1
- "$ACCUMULO_HOME"/bin/accumulo shell -u "$ACCUMULO_USER" -p "$ACCUMULO_PASSWORD" "$@"
+ if [[ $ACCUMULO_VERSION =~ ^[23]\..*$ ]]; then
+ "$ACCUMULO_HOME"/bin/accumulo shell -u "$ACCUMULO_USER" -p "$ACCUMULO_PASSWORD" "$@"
+ else
+ "$ACCUMULO_HOME"/bin/accumulo shell "$@"
+ fi
Review Comment:
Good idea. Done in 6972c36
--
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] Support Accumulo 4 in uno ashell [accumulo-uno]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on code in PR #316:
URL: https://github.com/apache/accumulo-uno/pull/316#discussion_r3769682997
##########
bin/impl/commands.sh:
##########
@@ -287,7 +287,11 @@ function uno_status_main() {
function uno_ashell_main() {
check_dirs ACCUMULO_HOME || return 1
- "$ACCUMULO_HOME"/bin/accumulo shell -u "$ACCUMULO_USER" -p "$ACCUMULO_PASSWORD" "$@"
+ if [[ $ACCUMULO_VERSION =~ ^[23]\..*$ ]]; then
+ "$ACCUMULO_HOME"/bin/accumulo shell -u "$ACCUMULO_USER" -p "$ACCUMULO_PASSWORD" "$@"
+ else
+ "$ACCUMULO_HOME"/bin/accumulo shell "$@"
+ fi
Review Comment:
This should work with 2.1 also. We should just drop the use of these properties entirely.
--
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: commits-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org