(accumulo-access) branch main updated: Removed public modifier from constructors (#109)

[ctubbsii]

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

ctubbsii 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 68a0b69  Removed public modifier from constructors (#109)
68a0b69 is described below

commit 68a0b6910d5e2fd3e306521ab212e49437df8e80
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Wed Mar 11 14:15:29 2026 -0400

    Removed public modifier from constructors (#109)
    
    Removed public modifier from AccessEvaluatorImpl
    and AccessExpressionImpl constructors to force
    a single entry point through AccessImpl. Unable
    to remove public modifier from AuthorizationsImpl
    constructor as it's a record type.
---
 .../java/org/apache/accumulo/access/impl/AccessEvaluatorImpl.java     | 4 ++--
 .../java/org/apache/accumulo/access/impl/AccessExpressionImpl.java    | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessEvaluatorImpl.java
 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessEvaluatorImpl.java
index 3912077..e4c5374 100644
--- 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessEvaluatorImpl.java
+++ 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessEvaluatorImpl.java
@@ -41,7 +41,7 @@ public final class AccessEvaluatorImpl implements 
AccessEvaluator {
   /**
    * Create an AccessEvaluatorImpl using an Authorizer object
    */
-  public AccessEvaluatorImpl(Predicate<String> authorizationChecker,
+  AccessEvaluatorImpl(Predicate<String> authorizationChecker,
       AuthorizationValidator authorizationValidator) {
     this.authorizedPredicate = auth -> 
authorizationChecker.test(auth.toString());
     this.authorizationValidator = authorizationValidator;
@@ -50,7 +50,7 @@ public final class AccessEvaluatorImpl implements 
AccessEvaluator {
   /**
    * Create an AccessEvaluatorImpl using a collection of authorizations
    */
-  public AccessEvaluatorImpl(Authorizations authorizations,
+  AccessEvaluatorImpl(Authorizations authorizations,
       AuthorizationValidator authorizationValidator) {
     var authsSet = authorizations.asSet();
     final Set<CharsWrapper> wrappedAuths = new HashSet<>(authsSet.size());
diff --git 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
index 2668152..75de7ee 100644
--- 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
+++ 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
@@ -31,7 +31,7 @@ public final class AccessExpressionImpl extends 
AccessExpression {
   private final String expression;
   private final AtomicReference<ParsedAccessExpression> parseTreeRef = new 
AtomicReference<>();
 
-  public AccessExpressionImpl(String expression) {
+  AccessExpressionImpl(String expression) {
     this.expression = expression;
   }