You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by mb...@apache.org on 2014/10/02 03:35:12 UTC
[2/3] git commit: HBASE-12120 HBase shell doesn't allow deleting of a
cell by user with W-only permissions to it (Srikanth Srungarapu)
HBASE-12120 HBase shell doesn't allow deleting of a cell by user with W-only permissions to it (Srikanth Srungarapu)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e00492b2
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e00492b2
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e00492b2
Branch: refs/heads/branch-1
Commit: e00492b2c68b8e09510097a976a4c0d6178c33d4
Parents: 947f09a
Author: Matteo Bertozzi <ma...@cloudera.com>
Authored: Thu Oct 2 02:31:02 2014 +0100
Committer: Matteo Bertozzi <ma...@cloudera.com>
Committed: Thu Oct 2 02:32:25 2014 +0100
----------------------------------------------------------------------
hbase-shell/src/main/ruby/hbase/table.rb | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/e00492b2/hbase-shell/src/main/ruby/hbase/table.rb
----------------------------------------------------------------------
diff --git a/hbase-shell/src/main/ruby/hbase/table.rb b/hbase-shell/src/main/ruby/hbase/table.rb
index cc4f2a7..a83c56a 100644
--- a/hbase-shell/src/main/ruby/hbase/table.rb
+++ b/hbase-shell/src/main/ruby/hbase/table.rb
@@ -169,7 +169,11 @@ EOF
# Delete a row
def _deleteall_internal(row, column = nil,
timestamp = org.apache.hadoop.hbase.HConstants::LATEST_TIMESTAMP, args = {})
- raise ArgumentError, "Row Not Found" if _get_internal(row).nil?
+ # delete operation doesn't need read permission. Retaining the read check for
+ # meta table as a part of HBASE-5837.
+ if is_meta_table?
+ raise ArgumentError, "Row Not Found" if _get_internal(row).nil?
+ end
temptimestamp = timestamp
if temptimestamp.kind_of?(Hash)
timestamp = org.apache.hadoop.hbase.HConstants::LATEST_TIMESTAMP