[GitHub] [accumulo] ctubbsii commented on issue #1643: Rename master references in public API

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #1643:
URL: https://github.com/apache/accumulo/issues/1643#issuecomment-690379138


   @jkosh44 The main issue here is to address the user-facing public APIs, because require thoughtful consideration with a plan, and affect our versioning, and are not simple renames. Those should go through a deprecation cycle (with replacements), and be dropped on the next major version.
   
   Take a look at our public API definition at https://accumulo.apache.org/api/ for details on what is considered public API.
   
   There are (at least) two separate, but related, steps that can be done also, but may be less risky, or at least, can be worked independently:
   
   1. User-facing non-API things like documentation, configuration properties, and configuration files, and the monitor
   2. Everything else that is internal-only, including things like `MasterClient`, field names (with careful consideration of serialization), and test cases, can probably be changed at any time, because they have no impact on the end user.
   
   These tasks should be performed separately, and be made incrementally so as to reduce the risk of mistakes, and to not overwhelm reviewers (because the changes will be large).


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org