keith-turner opened a new pull request, #96:
URL: https://github.com/apache/accumulo-access/pull/96
This is still a work in progress, but its close to being complete. Modified
the Accumulo Access code to only use String and unicode. Made the following
changes.
* Removed all methods that take `byte[]` in the API
* Removed the many static entry points in the API and replaced them w/ a
single static entry point.
* Reworked the internal code to operate on char instead of byte.
* Limited to authorizations to valid unicode characters that are not ISO
control characters by default. This is configurable though.
Realized it would be really nice if users could further limit authorizations
to a smaller subset of characters while working on this. The existing Accumulo
Access APIs had a ton of static entry points and there was no one place that
configurable behavior could be introduced . That led to reworking the API to
have a single entry point where configuration could be applied instead of the
many entry static points. Currently this entry point only allows setting a
configurable authorization validator, but it makes it easy to support future
configuration. This new single entry point is conceptually similar to how Gson
works, it was inspired by that.
Still puzzling about how Accumulo might use this code or if that is
possible. Want to experiment w/ that some.
Have not yet run the benchmarks w/ these code changes.
fixes #88
--
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]
