apache / accumulo-website

Apache Accumulo Website
https://accumulo.apache.org
14 stars 100 forks source link

Per table crypto documentation updates #342

Closed cshannon closed 1 year ago

cshannon commented 1 year ago

Here are in progress updated for the documentation for per table crypto. Creating a draft pull request for now so it can be reviewed for errors.

cshannon commented 1 year ago

@milleruntime and @dlmarion - I had some time tonight so I started working on the documentation updates for crypto. I still need to do some more polishing and fill in some more details but feel free to take a look and see what you think so far. The main doc changes so far should be good enough at this point to set up instance wide or per table crypto now with the new properties.

cshannon commented 1 year ago

I forgot to add I may have time to work on this PR a little bit more Wednesday, otherwise I plan to finish it up on Friday morning later this week so if you have any feedback before then that would be helpful.

cshannon commented 1 year ago

This file is generated in the main Accumulo build (see the warning at the top of the file) _docs-2/configuration/server-properties.md. I wasn't sure if you copied it or manually edited but it should just be copied from the latest in core/target/generated-docs/. If we see something wrong with the properties, then we can edit it in Accumulo main. I think they are all just pulled from Property.java.

I manually edited it quickly it as I just did a search and saw it and didn't realize it was generated, but this makes sense considering how large the properties are. So I can just grab it from the generated docs for my next update to this

cshannon commented 1 year ago

I've made all the requested changes so this can be reviewed again and I can make any further updates.