apache/incubator-storm — pull request #238 by rick-kilgore (2014-08-21 01:29:50+00:00)
Title: 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.

Note: I can't figure out how to run the automated tests on my laptop. I get an error like this:
java.lang.RuntimeException: backtype.storm.multilang.NoOutputException: Pipe to subprocess seems to be broken! No output read.