keith-turner commented on PR #4920: URL: https://github.com/apache/accumulo/pull/4920#issuecomment-2368906449
> Should we set the forRemoval flag to be true in the deprecated annotation? I guess if that's the case we'd be expected to remove it for sure from 4.0 so maybe we leave it as is and be more conservative. forRemoval has subtle problems and there was a PR that outlined those problems and removed its partial use in the code, but not sure what the PR number is. For this particular code its unlikely users reference it any other way than configuring it, so that leaves the log message as the primary way of notifying the user of it deprecation. I mainly added the deprecation annotation to avoid new direct uses in the accumulo code base. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
