-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22437/
-----------------------------------------------------------

Review request for accumulo.


Bugs: ACCUMULO-2887
    https://issues.apache.org/jira/browse/ACCUMULO-2887


Repository: accumulo


Description
-------

The logic for maintaining and notifying observers in NamespaceConfiguration and 
TableConfiguration is refactored into a new ObservableConfiguration class.


Diffs
-----

  core/src/main/java/org/apache/accumulo/core/conf/ObservableConfiguration.java 
PRE-CREATION 
  
core/src/test/java/org/apache/accumulo/core/conf/ObservableConfigurationTest.java
 PRE-CREATION 
  
server/base/src/main/java/org/apache/accumulo/server/conf/NamespaceConfWatcher.java
 7f3d73e 
  
server/base/src/main/java/org/apache/accumulo/server/conf/NamespaceConfiguration.java
 eab198e 
  
server/base/src/main/java/org/apache/accumulo/server/conf/TableConfWatcher.java 
c407309 
  
server/base/src/main/java/org/apache/accumulo/server/conf/TableConfiguration.java
 909b450 

Diff: https://reviews.apache.org/r/22437/diff/


Testing
-------

Unit tests pass, including the new ObservableConfigurationTest. I was unable to 
add observer testing for the refactored classes, but that will arrive late 
(probably under ACCUMULO-2615).


Thanks,

Bill Havanki

Reply via email to