ddanielr opened a new pull request, #3274:
URL: https://github.com/apache/accumulo/pull/3274
This PR is part of #3208 with the intended goal to be removal of the
`ManagerMetadataUtil` and `MetadataTableUtil` classes to consolidate metadata
operations.
The first part of the `ManagerMetadataUtil` class removal was completed in
#3255.
Currently, all methods have been refactored out from ManagerMetadataUtil and
that class has been removed.
However, these changes are currently failing the
`org.apache.accumulo.server.manager.state.RootTabletStateStoreTest.testRootTabletStateStore`
test due to a `NullPointerException` being thrown in `MockServerContext.java`
After poking at it, I've confirmed that the error is happening before my
code changes in `RootTabletStateStoreTest`.
It seems that the exception is thrown when attempting to access
configuration properties that should be set in the mock object.
`server/base/src/test/java/org/apache/accumulo/server/MockServerContext.java:47`
```
expect(context.getConfiguration().get(Property.INSTANCE_VOLUMES)).andReturn("file:///")
.anyTimes();
```
Any help would be appreciated.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
