keith-turner commented on code in PR #112:
URL: https://github.com/apache/accumulo-access/pull/112#discussion_r2925967176
##########
modules/core/src/main/java/org/apache/accumulo/access/Authorizations.java:
##########
@@ -28,6 +29,6 @@
*
* @since 1.0.0
*/
-public interface Authorizations extends Iterable<String> {
+public interface Authorizations extends Iterable<String>, Externalizable {
Review Comment:
Reading that does sound like it would work to me. Sounds like it checks for
instance of Externalizable first and then if that is not true then checks for
instance of Serializable. W/ this proposed change instance of Externalizable
would return true. Also the Externalizable type extends Serializable, so
everything that is Externalizable is also Serializable.
--
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]
