[IMPALA-6854] TestExchangeDelays::test_exchange_small_delay()'s implementation doesn't match description

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Won't Fix
• Affects Version/s: Impala 3.1.0
• Component/s: Backend
• Epic Color: ghx-label-2
• Assignee: Joe McDonnell
• Reporter: Joe McDonnell
• Created: 13/Apr/18 22:11
• Updated: 07/Feb/19 22:44
• Resolved: 07/Feb/19 22:44

Description

test_exchange_small_delay() has this description:

def test_exchange_small_delay(self, vector):
  """Test delays in registering data stream receivers where the first one or two
  batches will time out before the receiver registers, but subsequent batches will
  arrive after the receiver registers. Before IMPALA-2987, this scenario resulted in
  incorrect results.
  """

However, looking at the logs when the first sender times out, the query is cancelled. In my manual tests, the query does not return a result. This test case may be out of date. As it is, this overlaps with what test_exchange_large_delay() is doing.