(accumulo-access) branch main updated: Implement Serializable and Iterable where appropriate (#73)

[dlmarion]

This is an automated email from the ASF dual-hosted git repository.

dlmarion pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-access.git


The following commit(s) were added to refs/heads/main by this push:
     new dc4bed4  Implement Serializable and Iterable where appropriate (#73)
dc4bed4 is described below

commit dc4bed4f0e5416d57f77aef9a6bf2549e8ef78ca
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Mon Jul 8 13:46:11 2024 -0400

    Implement Serializable and Iterable where appropriate (#73)
---
 .../java/org/apache/accumulo/access/AccessExpression.java     |  4 +++-
 .../java/org/apache/accumulo/access/AccessExpressionImpl.java |  2 ++
 src/main/java/org/apache/accumulo/access/Authorizations.java  | 11 ++++++++++-
 3 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/accumulo/access/AccessExpression.java 
b/src/main/java/org/apache/accumulo/access/AccessExpression.java
index 84d8df9..360fc7f 100644
--- a/src/main/java/org/apache/accumulo/access/AccessExpression.java
+++ b/src/main/java/org/apache/accumulo/access/AccessExpression.java
@@ -18,6 +18,8 @@
  */
 package org.apache.accumulo.access;
 
+import java.io.Serializable;
+
 /**
  * This class offers the ability to validate, build, and normalize access 
expressions. An instance
  * of this class should wrap an immutable, validated access expression. If 
passing access
@@ -77,7 +79,7 @@ package org.apache.accumulo.access;
  * @see <a href="https://github.com/apache/accumulo-access";>Accumulo Access 
Documentation</a>
  * @since 1.0.0
  */
-public interface AccessExpression {
+public interface AccessExpression extends Serializable {
 
   /**
    * @return the expression that was used to create this object.
diff --git a/src/main/java/org/apache/accumulo/access/AccessExpressionImpl.java 
b/src/main/java/org/apache/accumulo/access/AccessExpressionImpl.java
index 5c549b8..d91c388 100644
--- a/src/main/java/org/apache/accumulo/access/AccessExpressionImpl.java
+++ b/src/main/java/org/apache/accumulo/access/AccessExpressionImpl.java
@@ -25,6 +25,8 @@ import java.util.function.Predicate;
 
 final class AccessExpressionImpl implements AccessExpression {
 
+  private static final long serialVersionUID = 1L;
+
   public static final AccessExpression EMPTY = new AccessExpressionImpl("", 
false);
 
   private final String expression;
diff --git a/src/main/java/org/apache/accumulo/access/Authorizations.java 
b/src/main/java/org/apache/accumulo/access/Authorizations.java
index eb333a7..407b2eb 100644
--- a/src/main/java/org/apache/accumulo/access/Authorizations.java
+++ b/src/main/java/org/apache/accumulo/access/Authorizations.java
@@ -18,6 +18,8 @@
  */
 package org.apache.accumulo.access;
 
+import java.io.Serializable;
+import java.util.Iterator;
 import java.util.Set;
 
 /**
@@ -31,7 +33,9 @@ import java.util.Set;
  *
  * @since 1.0.0
  */
-public final class Authorizations {
+public final class Authorizations implements Iterable<String>, Serializable {
+
+  private static final long serialVersionUID = 1L;
 
   private static final Authorizations EMPTY = new Authorizations(Set.of());
 
@@ -90,4 +94,9 @@ public final class Authorizations {
       return new Authorizations(authorizations);
     }
   }
+
+  @Override
+  public Iterator<String> iterator() {
+    return authorizations.iterator();
+  }
 }