keith-turner commented on code in PR #6146:
URL: https://github.com/apache/accumulo/pull/6146#discussion_r2848462819
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/ScanServer.java:
##########
@@ -966,9 +1043,9 @@ public InitialScan startScan(TInfo tinfo, TCredentials
credentials, TKeyExtent t
KeyExtent extent = getKeyExtent(textent);
- if (extent.isMeta() && !isSystemUser(credentials)) {
- throw new TException(
- "Only the system user can perform eventual consistency scans on the
root and metadata tables");
+ if (!isAllowed(extent.tableId())) {
+ throw new TException("Scan of table " + extent.tableId() + " disallowed
by property: "
Review Comment:
startMultiScan was changed to throw a thrift security exception, can the
same change be made for startScan?
--
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]
