On Mon, Nov 28, 2011 at 8:26 AM, Doug Meil <[email protected]> wrote: > Hi folks- > > In Compression.java there is this comment… > > > /** > > * Compression algorithms. The ordinal of these cannot change or else you > > * risk breaking all existing HFiles out there. Even the ones that are > > * not compressed! (They use the NONE algorithm) > > */ > > public static enum Algorithm { > > … which makes the prospect of adding new instances slightly terrifying. Has > anybody considered changing the constructor to supply an ordinal value, and > that way if anybody adds a custom one they could supply a value like 9999999 > or something hopefully never used by HBase? > >
The comment is just to warn against damage if ordinals are removed or reordered. Do whatever it is you need to make it work Doug. It should be possible to insert other compressors w/o gymnastics. St.Ack
