You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/04/08 15:06:43 UTC

[GitHub] [accumulo] keith-turner opened a new issue, #2619: MiniAccumuloCluster.getClientProperties() returns a mutable cached object

keith-turner opened a new issue, #2619:
URL: https://github.com/apache/accumulo/issues/2619

   **Describe the bug**
   
   As seen [here](https://github.com/apache/accumulo/blob/d454afd39c91ee0a1ab267e0caafc52254d209b0/minicluster/src/main/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImpl.java#L840-L847) and [here](https://github.com/apache/accumulo/blob/d454afd39c91ee0a1ab267e0caafc52254d209b0/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloCluster.java#L168) mini accumulo cluster returns a mutable Properties object that it caches.  I noticed this while reviewing scan server test that would get the properties and mutate them, which could lead to interference between test.  However this may be a more general problem that could lead to subtle unexpected behavior.
   
   **Expected behavior**
   
   `getClientProperties()` should either return an immutable object or create a new object each time its called.


-- 
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: notifications-unsubscribe@accumulo.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo] ctubbsii closed issue #2619: MiniAccumuloCluster.getClientProperties() returns a mutable cached object

Posted by GitBox <gi...@apache.org>.
ctubbsii closed issue #2619: MiniAccumuloCluster.getClientProperties() returns a mutable cached object
URL: https://github.com/apache/accumulo/issues/2619


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org