search

apache / accumulo-website

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

Expand note about CompactionStrategy #297

Closed milleruntime closed 2 years ago

ctubbsii commented 2 years ago

@milleruntime Should this be put in the next-release branch instead of main? That way, it gets merged when we're ready to release 2.1? (If so, click "EDIT", and change the base branch for this PR from the drop-down below the PR title, before merging.)

milleruntime commented 2 years ago

@milleruntime Should this be put in the next-release branch instead of main? That way, it gets merged when we're ready to release 2.1? (If so, click "EDIT", and change the base branch for this PR from the drop-down below the PR title, before merging.)

I think it is OK to get merged since it is just going in the release notes. I was leaving it open so @keith-turner wanted to give it a look.

ctubbsii commented 2 years ago

I think it is OK to get merged since it is just going in the release notes. I was leaving it open so @keith-turner wanted to give it a look.

Good point. I remember now why I didn't make this comment when I first reviewed this 2 days ago. :smiley_cat: