+ 1 LGTM
As for commenting out – I think I like this idea. We are probably going to enable snappy in Bigtop at some point so that'll be pretty useful.
The default values for io.compression.codes already cover the ones that are freely available:
Would this be better commented out, so it can optionally be enabled ?
So there is no value in keeping the gzip around?