Skip to content

Updating commands in spark, reservations, terasort, and uniquecols examples#138

Open
Amemeda wants to merge 3 commits into
apache:mainfrom
Amemeda:final-command-updates
Open

Updating commands in spark, reservations, terasort, and uniquecols examples#138
Amemeda wants to merge 3 commits into
apache:mainfrom
Amemeda:final-command-updates

Conversation

@Amemeda

@Amemeda Amemeda commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

reservations:

  • "$ /path/to/accumulo org.apache.accumulo.server.util.ListInstances" updated to "$ accumulo inst list-instances"
  • updated usages of "-u root -p secret -e" to "--user root --password secret --execute-command"

terasort:

  • updated usages of "-u root -p secret -e" to "--user root --password secret --execute-command"

uniquecols:

  • updated "$ /path/to/accumulo shell -u username -p secret" to "$ accumulo shell --user root --password secret"
  • updated multiple references of "examples" table that was misspelled as "example"

spark:

  • updated instructions specifying user to be in spark directory before running run.sh batch

Comment thread docs/reservations.md
Comment on lines +34 to +35
$ accumulo shell --user root --password secret --execute-command "createnamespace examples"
$ accumulo shell --user root --password secret --execute-command "createtable examples.ars"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can drop the user and password options, as the user can specify these in a client config file, and that's not what this example is trying to teach, so it's fine to omit them. I would not expand the execute-command, as it's much more common for people to use the single letter version.

Comment thread spark/README.md
Comment on lines +39 to +42
This application can be run using the following commands:

./run.sh batch /path/to/accumulo-client.properties
$ cd /path/to/accumulo-examples/spark
$ ./run.sh batch /path/to/accumulo-client.properties

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This README is already in the directory, so the cd is implied. This part doesn't really need to change, does it?

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