-
Notifications
You must be signed in to change notification settings - Fork 471
Documentation updates #190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
mikewalch
commented
Dec 6, 2016
- Removed old HTML documentation. Did work in multiple commits to simplify review.
- Improved docs of examples
* Moved documentation to clients chapter were some already existed
|
Gave it a glance, looks fine to me! |
| | [terasort] | Generating random data and sorting it using Accumulo. | | ||
| | [visibility] | Using visibilities (or combinations of authorizations). Also shows user permissions. | | ||
|
|
||
| [1]: https://accumulo.apache.org/1.5/accumulo_user_manual#_user_administration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know you are just copying, but I noticed this was the 1.5 user manual
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linked to latest user manual in 66f02d3
| See the License for the specific language governing permissions and | ||
| limitations under the License. | ||
| --> | ||
| <html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For all of these html files do you know if the content is in the user manual?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For every HTML file, similar content existed in user manual. However, there were a few cases where not all text HTML doc was not moved to user manual. I tried to add this text. However, I did not add old JMX metrics docs in metrics.html as it looks like hadoop metrics is preferred. Also, administration.html could be checked over too. While it looks like all docs were added to manual, there was a lot of documentation so it was not easy to confirm.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SGTM
|
|
||
| A complete example of using Bulk Ingest can be found at | ||
| +accumulo/docs/examples/README.bulkIngest+. | ||
| A complete example of using Bulk Ingest can be found at +docs/examples/bulkIngest.md+. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice to fix these locations
* Renamed files and structured them to be markdown * Fixed references in user manual
* Similar docs exist in 'Table Configuration' section of user manual
* Most documentation exists in manual but added some missing text
…racing.html * Similar docs exist in 'Administration' section of user manual
* Similar documentation exists in 'High-speed Ingest' section of user manual
66f02d3 to
c0e68a3
Compare