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 e77f80d Remove unused constructor, fix warnings (#87)
e77f80d is described below
commit e77f80d4d8d7cdebef6dad064638b84f7befebe4
Author: Dave Marion <[email protected]>
AuthorDate: Tue Nov 18 07:23:58 2025 -0500
Remove unused constructor, fix warnings (#87)
---
.../java/org/apache/accumulo/access/AccessEvaluator.java | 1 -
.../java/org/apache/accumulo/access/AccessExpression.java | 5 ++++-
.../java/org/apache/accumulo/access/BytesWrapper.java | 15 ---------------
.../apache/accumulo/access/ParsedAccessExpression.java | 2 ++
4 files changed, 6 insertions(+), 17 deletions(-)
diff --git a/src/main/java/org/apache/accumulo/access/AccessEvaluator.java
b/src/main/java/org/apache/accumulo/access/AccessEvaluator.java
index d80f2e1..6857343 100644
--- a/src/main/java/org/apache/accumulo/access/AccessEvaluator.java
+++ b/src/main/java/org/apache/accumulo/access/AccessEvaluator.java
@@ -19,7 +19,6 @@
package org.apache.accumulo.access;
import java.util.Collection;
-import java.util.List;
/**
* This class is used to decide if an entity with a given set of
authorizations can access
diff --git a/src/main/java/org/apache/accumulo/access/AccessExpression.java
b/src/main/java/org/apache/accumulo/access/AccessExpression.java
index 6f0f18b..e9d8ab5 100644
--- a/src/main/java/org/apache/accumulo/access/AccessExpression.java
+++ b/src/main/java/org/apache/accumulo/access/AccessExpression.java
@@ -95,6 +95,9 @@ import java.util.function.Predicate;
* @since 1.0.0
*/
public abstract class AccessExpression implements Serializable {
+
+ private static final long serialVersionUID = 1L;
+
/*
* This is package private so that it can not be extended by classes outside
of this package and
* create a mutable implementation. In this package all implementations that
extends are
@@ -299,7 +302,7 @@ public abstract class AccessExpression implements
Serializable {
}
/**
- * Reverses what {@link #quote(String)} does, so will unquote an unescape an
authorization if
+ * Reverses what {@link #quote(String)} does, so will unquote and unescape
an authorization if
* needed. If the authorization is not quoted then it is returned as-is.
*
* @throws NullPointerException when the argument is null
diff --git a/src/main/java/org/apache/accumulo/access/BytesWrapper.java
b/src/main/java/org/apache/accumulo/access/BytesWrapper.java
index db6d48d..f9b53bb 100644
--- a/src/main/java/org/apache/accumulo/access/BytesWrapper.java
+++ b/src/main/java/org/apache/accumulo/access/BytesWrapper.java
@@ -40,21 +40,6 @@ final class BytesWrapper implements Comparable<BytesWrapper>
{
set(data, 0, data.length);
}
- /**
- * Creates a new sequence from a subsequence of the given byte array. The
given byte array is used
- * directly as the backing array, so later changes made to the (relevant
portion of the) array
- * reflect into the new sequence.
- *
- * @param data byte data
- * @param offset starting offset in byte array (inclusive)
- * @param length number of bytes to include in sequence
- * @throws IllegalArgumentException if the offset or length are out of
bounds for the given byte
- * array
- */
- BytesWrapper(byte[] data, int offset, int length) {
- set(data, offset, length);
- }
-
byte byteAt(int i) {
return data[offset + checkIndex(i, length)];
}
diff --git
a/src/main/java/org/apache/accumulo/access/ParsedAccessExpression.java
b/src/main/java/org/apache/accumulo/access/ParsedAccessExpression.java
index 9f15fbe..a51e8ab 100644
--- a/src/main/java/org/apache/accumulo/access/ParsedAccessExpression.java
+++ b/src/main/java/org/apache/accumulo/access/ParsedAccessExpression.java
@@ -30,6 +30,8 @@ import java.util.List;
*/
public abstract class ParsedAccessExpression extends AccessExpression {
+ private static final long serialVersionUID = 1L;
+
/*
* This is package private so that it can not be extended by classes outside
of this package and
* create a mutable implementation. In this package all implementations that
extends are
