accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Vines" <>
Subject Review Request 17468: ACCUMULO-1479 - simplification of namespace SecurityOperation changes
Date Tue, 28 Jan 2014 21:43:54 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for accumulo.

Repository: accumulo


This is a refactor of the SecurityOperation changes done by the namespace implementation.
Root reason being to simplify the changes to the can____ operations to prevent them from being
overly complex. Surrounding changes were made to add Namespace permissions for bulk import
and dropping the namespace. Additionally, some other changes were made around the Create fate
operation to minimize the number of namespaceId lookups, as well as to do namespace checking
before security checking, which could have resulted in strange errors to clients.


  core/src/main/java/org/apache/accumulo/core/security/ 1066bc4 
  server/base/src/main/java/org/apache/accumulo/server/security/ 4b302f0

  server/master/src/main/java/org/apache/accumulo/master/ afcda86 
  server/master/src/main/java/org/apache/accumulo/master/tableOps/ 9535781

  test/src/main/java/org/apache/accumulo/test/randomwalk/security/ 16310a5

  test/src/test/java/org/apache/accumulo/test/ addb377 



All tests, including ITs, run successfully.


John Vines

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message