[BUG] S3 Source poll delay will always call Thread.sleep() if poll delay is greater than 0

[asifsmohammed]

Describe the bug
When we process a batch of SQS messages, we wait for some time before processing next batch. The wait time is based on number of messages processed and poll time. If poll time is greater than 0 and maximum messages is default value (which is 10 and it's the maximum value) SQS worker will quite likely have a delay because SQS might never return maximum number of messages.
SQS will return as soon as it gets at least one message even if maximum messages is set to 10.

if (messagesProcessed < sqsOptions.getMaximumMessages() && s3SourceConfig.getSqsOptions().getPollDelay().toMillis() > 0) {

Expected behavior
SQS should wait if at least one message is processed instead of checking against maximum messages.

[dlvenable]

It might not be fair to say that it will always delay. :) But, it appears quite likely that it often will introduce the delay.