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 2012/07/03 01:27:35 UTC
svn commit: r1356517 -
/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java
Author: apurtell
Date: Mon Jul 2 23:27:34 2012
New Revision: 1356517
URL: http://svn.apache.org/viewvc?rev=1356517&view=rev
Log:
HBASE-6303. HCD.setCompressionType should use Enum support for storing compression types as strings
Modified:
hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java
Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java?rev=1356517&r1=1356516&r2=1356517&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java Mon Jul 2 23:27:34 2012
@@ -595,14 +595,7 @@ public class HColumnDescriptor implement
* @return this (for chained invocation)
*/
public HColumnDescriptor setCompressionType(Compression.Algorithm type) {
- String compressionType;
- switch (type) {
- case LZO: compressionType = "LZO"; break;
- case GZ: compressionType = "GZ"; break;
- case SNAPPY: compressionType = "SNAPPY"; break;
- default: compressionType = "NONE"; break;
- }
- return setValue(COMPRESSION, compressionType);
+ return setValue(COMPRESSION, type.getName().toUpperCase());
}
/** @return data block encoding algorithm used on disk */