You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by an...@apache.org on 2013/12/22 19:19:26 UTC

svn commit: r1553013 - /hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java

Author: anoopsamjohn
Date: Sun Dec 22 18:19:26 2013
New Revision: 1553013

URL: http://svn.apache.org/r1553013
Log:
HBASE-10223 [VisibilityController] cellVisibility presence check on Delete mutation is wrong

Modified:
    hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java

Modified: hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java?rev=1553013&r1=1553012&r2=1553013&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java (original)
+++ hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java Sun Dec 22 18:19:26 2013
@@ -683,7 +683,7 @@ public class VisibilityController extend
             }
           }
         }
-      } else {
+      } else if (cellVisibility != null) {
         // CellVisibility in a Delete is not legal! Fail the operation
         miniBatchOp.setOperationStatus(i, new OperationStatus(SANITY_CHECK_FAILURE,
             "CellVisibility cannot be set on Delete mutation"));