ctubbsii commented on PR #3977:
URL: https://github.com/apache/accumulo/pull/3977#issuecomment-1841907728

   > I keep getting conflicting advice on whether to use the 'forRemoval' 
attribute. 
   
   I'm not sure who is in favor of it, but I imagine whoever is, it's because 
it's helpful to be explicit. I suspect Java added it because so many people 
mark things deprecated and then they stayed deprecated forever in some Java 
libraries. But that's not really necessary in a world where semantic versioning 
is used and major releases are expected for breaking changes. I have personally 
avoided it because of the problems that it causes but I haven't really objected 
to using it in principal. Any objections that I might have her purely practical.
   
   > Maybe we can decide at a project level?
   
   The options would be to use it everywhere, use it nowhere, not care if it's 
used or not, or develop some criteria by which we decide when it should be used 
and when it shouldn't. Since semantic versioning drives us, I don't really 
think that last option is useful. The third option can lead to confusion. But I 
think for practical reasons using it everywhere isn't really viable. So that 
really only leaves use it nowhere. But we can discuss and decide on the mailing 
list.
   


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

Reply via email to