ctubbsii commented on a change in pull request #744: Fixes #533 - Add scan
authorization to Accumulo clients
URL: https://github.com/apache/accumulo/pull/744#discussion_r230501338
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/impl/AccumuloClientImpl.java
##########
@@ -125,6 +125,13 @@ public BatchScanner createBatchScanner(String tableName,
Authorizations authoriz
return createBatchScanner(tableName, authorizations, numQueryThreads);
}
+ @Override
+ public BatchScanner createBatchScanner(String tableName) throws
TableNotFoundException,
+ AccumuloSecurityException, AccumuloException {
+ Authorizations auths =
securityOperations().getUserAuthorizations(context.getPrincipal());
Review comment:
See comment above. This is okay for a first pass implementation, but not
sufficient with pluggable authorization providers which could have an infinite
set of auths.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
