search

mozilla-metrics / akela

A bunch of utility classes for Java, Hadoop, HBase, Pig, etc.
Apache License 2.0
76 stars 31 forks source link

MultiScanTableMapReduceUtil accidentally skips 1/256 of output because it ignores byte 127. #6

Closed bsmedberg closed 11 years ago

bsmedberg commented 11 years ago

@harshach/@reid_write This class is being used for pydoop, but I suspect changes to it could also affect numbers from a bunch of other mapreduce jobs. I think we're accidentally leaving out one random class from all of our current mapreduce jobs for telemetry and FHR at least.

mreid-moz commented 11 years ago

Nice catch, thanks. I'll add a test case presently. Note that this doesn't affect FHR since it doesn't use salted keys.