keith-turner commented on PR #112:
URL: https://github.com/apache/accumulo-access/pull/112#issuecomment-4048449653

   >  it might be better / easier to just remove Serializable from 
AccessExpression and Authorizations. 
   
   One nice thing about that from an API evolution perspective is its much much 
easier to add that later than it would be to remove it later.  So if we do not 
have Serializable for 1.0, we can always easily add it for 1.1.  So I would 
also be in favor of removing Serializable for now.  We also have not test for 
it, if we are going to implement the interface we should have regression test 
for the functionality.


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