I agree with your general reasoning, but make sure that it is documented. One would _likely_ toss out unmapped reads before using this partitioner, but if they are unfamiliar with what key skew entails, that could take them by surprise.