Skip to content

Updated 2 commands in Bloom Filter Example that were outdated#130

Merged
DomGarguilo merged 3 commits into
apache:mainfrom
Amemeda:bloom-filters-command-fixes
Jun 3, 2026
Merged

Updated 2 commands in Bloom Filter Example that were outdated#130
DomGarguilo merged 3 commits into
apache:mainfrom
Amemeda:bloom-filters-command-fixes

Conversation

@Amemeda

@Amemeda Amemeda commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

2 commands in bloom.md were outdated

$ accumulo shell -u username -p password -e 'tables -l' (-u and -p updated to --user --password respectively, filled actual username and password)

$ accumulo rfile-info /accumulo/tables/3/default_tablet/F00000dj.rf (missing keyword "file" after "accumulo")

@DomGarguilo

Copy link
Copy Markdown
Member

This looks good to me.

I fixed the build in #129 so you'll need to merge the main branch into this one before the checks are able to pass here. Once that is passing though I think this can be merged.

As a follow on, I'm sure there are a lot of other spots that could use this same cleanup which would be good to fix as well.

@DomGarguilo DomGarguilo merged commit d5a3c8d into apache:main Jun 3, 2026
1 check passed
@DomGarguilo

Copy link
Copy Markdown
Member

Thanks for the PR @Amemeda . I see you aren't listed on our "people" page, acknowledging contributors. If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/main/pages/people.md and leave a reference to accumulo-examples/pull/130 in your commit log.

If you intend to be a regular contributor to Accumulo projects, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself.

(this message has been mostly copied and pasted from this comment)

Amemeda added a commit to Amemeda/accumulo-website that referenced this pull request Jun 5, 2026
DomGarguilo pushed a commit to apache/accumulo-website that referenced this pull request Jun 5, 2026
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