Skip to content

Updated commands in Delete Key Value Pair and Combiner that were outdated#134

Merged
DomGarguilo merged 7 commits into
apache:mainfrom
Amemeda:4.x
Jun 9, 2026
Merged

Updated commands in Delete Key Value Pair and Combiner that were outdated#134
DomGarguilo merged 7 commits into
apache:mainfrom
Amemeda:4.x

Conversation

@Amemeda

@Amemeda Amemeda commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Combiner:

  • $ bin/accumulo shell -u username" updated to "$ accumulo shell -u root"
  • added "username@instance examples.runners> config -t examples.runners -s table.class.loader.context=file:///path/to/accumulo-examples/target/accumulo-examples.jar" before setiter command

Delete Key Value Pair

  • "$ /path/to/accumulo shell -u username -p secret" up updated to "$ accumulo shell --user root --password secret" in 3 locations
  • "$ /path/to/accumulo rfile-info -d hdfs://localhost/accumulo/tables/1t/default_tablet/F00007em.rf" updated to "$ accumulo file rfile-info -d hdfs://localhost/accumulo/tables/1t/default_tablet/F00007em.rf" in 3 locations

@DomGarguilo

Copy link
Copy Markdown
Member

For the combiner example, I'm actually not sure that we need that added line to set the jar as the classloader context. I think that is contained in the jar that we copy over into the lib dir as outline in the setup steps.

For the combiner I ran through the example and things worked when I followed the doc with these exceptions:

  1. I had to restart my accumulo instance so the jar was picked up by the tservers (maybe this could be pointed out in the description of this doc)
  2. I did not do the config -t examples.runners -s table.class.loader.context= step

@DomGarguilo

Copy link
Copy Markdown
Member

I'm not too sure why the combiner doc is just a copy and paste of the whole shell as opposed to the commands like all the other docs. Probably not worth changing that here but we should update the version: 2.1.0-SNAPSHOT

Comment thread docs/deleteKeyValuePair.md Outdated
Comment thread docs/deleteKeyValuePair.md Outdated
@DomGarguilo DomGarguilo merged commit 9e338ec into apache:main Jun 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants