You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2014/08/14 20:45:27 UTC
[5/6] git commit: Amend HBASE-11438 [Visibility Controller] Support
UTF8 character as Visibility Labels; Fix failing tests
Amend HBASE-11438 [Visibility Controller] Support UTF8 character as Visibility Labels; Fix failing tests
String(byte[]) constructor may not select UTF8 aware conversion
depending on environment
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/a79e5ce5
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/a79e5ce5
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/a79e5ce5
Branch: refs/heads/branch-1
Commit: a79e5ce5250d76a367ca00c88a42d70530ad5f6b
Parents: 1f5ee71
Author: Andrew Purtell <ap...@apache.org>
Authored: Thu Aug 14 11:43:45 2014 -0700
Committer: Andrew Purtell <ap...@apache.org>
Committed: Thu Aug 14 11:45:59 2014 -0700
----------------------------------------------------------------------
.../apache/hadoop/hbase/security/visibility/ExpressionParser.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/a79e5ce5/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ExpressionParser.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ExpressionParser.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ExpressionParser.java
index bb17b48..cfbc97d 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ExpressionParser.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ExpressionParser.java
@@ -87,7 +87,7 @@ public class ExpressionParser {
// If we don't do this way then we may have to handle while checking the authorizations.
// Better to do it here.
byte[] array = com.google.common.primitives.Bytes.toArray(list);
- String leafExp = new String(array).trim();
+ String leafExp = Bytes.toString(array).trim();
if (leafExp.isEmpty()) {
throw new ParseException("Error parsing expression " + expS + " at column : " + index);
}