[GitHub] [accumulo] ctubbsii commented on issue #1640: Rename Accumulo master properties

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

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


   @brianloss I think it might be a bit burdensome to use `AccumuloConfiguration.resolve` in so many places. Also, it might get complicated if we are monitoring any properties for changes (I'm not sure if we are). As an alternative, I was thinking we could:
   
   1. Produce an error if both `master.*` and `manager.*` properties exist in the `SiteConfiguration` file on startup and fail to start,
   2. Automatically remap any `master.*` to `manager.*` when loading the `SiteConfiguration` file, issuing a warning if any `master.*` are detected, and
   3. On upgrade, rename any properties in ZooKeeper automatically.
   
   That way, we don't need to maintain any deprecated properties side-by-side in `Property.java` for this, or do anything special to deal with `AccumuloConfiguration.resolve`. We can simply rename the enums and property keys in `Property.java`.


----------------------------------------------------------------
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