ctubbsii opened a new pull request #2425:
URL: https://github.com/apache/accumulo/pull/2425
* Create a separate NativeMapLoader and relocate into that new class the
code to load the native libraries that was previously in NativeMap
alongside the actual code for handling native maps
* Remove unnecessarily redundant `isLoaded()` checks, since the behavior
is now to `System.exit(1)` if the native code is configured to be
used, but cannot be loaded. There is no longer a question of whether
they are loaded. They are always loaded if configured to be, or else
the tserver will die on startup.
* Simplify and organize the loader code
* Make the loader code work better with tests by providing a method
specifically for testing (for NativeMapIT, and InMemoryMapIT,
specifically, because they need to load native maps inside the test
thread, and not on a tserver)
* Update ConfigurableMacBase so it will ensure the native maps are built
before Mini is started, so the tests don't hang because the native
maps aren't built (only applies to ConfigurableMacBase tests... other
tests may need something similar)
--
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]
