Skip to content
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

Assess network port table in 2.x docs #402

Closed
DomGarguilo opened this issue Sep 1, 2023 · 4 comments · Fixed by #409
Closed

Assess network port table in 2.x docs #402

DomGarguilo opened this issue Sep 1, 2023 · 4 comments · Fixed by #409

Comments

@DomGarguilo
Copy link
Member

The table here: https://accumulo.apache.org/docs/2.x/administration/in-depth-install#network

Many of the entries in the table are deprecated or have broken links. It should be determined whether any should be removed or fixed.

image

@EdColeman
Copy link
Contributor

It may not be needed for this issue - but it may help if any changes considered what may be necessary to address changes coming down the line with elasticity changes, things like multiple-managers,...

One possibility would be a future change that wanted two columns for IP addresses, one 2.x, 3.x and a second for 4.0 that would map the changes necessary between versions? Mainly just speculating that there will be significant changes and that could possibly made easier by planning ahead. Or not...

@AlbertWhitlock
Copy link

Aren't some users not even on 2.1 yet? If that is true I think leaving the deprecated entries in makes sense (since that is when the replication code got deprecated. These users will be looking for the 2.x docs). But there is at least one broken link here, monitor.port.log4j.

@AlbertWhitlock
Copy link

Dom.....I can take this one.

@ctubbsii
Copy link
Member

I think all these were fixed by #409 . Can create a new issue or PR if there's more to do here.

@ctubbsii ctubbsii linked a pull request Dec 20, 2023 that will close this issue
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 a pull request may close this issue.

4 participants