ctubbsii commented on pull request #2414: URL: https://github.com/apache/accumulo/pull/2414#issuecomment-1018497378
> Oh, sorry, I thought that was tied to #2425 being completed. I likely have not had enough coffee and misread it. I can add that back in. Sorry for the confusion. It's parallel to what I did in the other PR. This wouldn't load the native maps, but it would do a sanity check to make sure that if the user set the cluster to have them enabled, they've at least provided the location to them so the mini has a good chance of loading them. We could also go further than what I suggested to check the contents of those, in order to at least look to see if a file exists. However, I'm reluctant to suggest Mini should actually load the native maps in that process, because mini is public API and that is the user's process we'd be loading a native library into, and we don't know the impact on the user that might cause. -- 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]
