fix issue on KafkaOffsetMetric
remove comment
don't discard unfinished messages that are not old

the ack() message in PartitionManager appears to have a bug that will
cause it to discard messages that have not been acknowledged even though
they have not fallen behind by maxOffsetBehind.

This can happen if there is at least one message that _is_ behind by
maxOffsetBehind.  The code correctly identifies that one or more
messages in _pending are old, but then potentially discards some that
are not.
external/storm-kafka: avoid NPE on null message payloads
Merge branch 'master' of github.com:rick-kilgore/incubator-storm into temp
Merge branch 'fixing_stormkafka' of github.com:fxjwind/incubator-storm into temp
Merge branch 'master' of github.com:relateiq/incubator-storm into temp
Add STORM-476, STORM-424 to changelog