GH-1646 Clear lastCommits after fixTxOffset

Resolves #1646

```
Failed to correct transactional offset(s)
java.lang.IllegalStateException: You can only check the position for partitions assigned to this consumer.
```

It is not clear how this happened since we remove revoked partitions from `lastCommits`.

However, `lastCommits` should be cleared, even if successful, to avoid unnecessary
processing if no records are received by the next poll.

- add the `lastCommits` to the error log
- clear `lastCommits` in a finally block

**cherry-pick to 2.5.x**

(cherry picked from commit ef6c115)

Author: garyrussell

spring-kafka/src/main/java/org/springframework/kafka/listener/KafkaMessageListenerContainer.java

@@ -1212,7 +1212,11 @@ private void fixTxOffsetsIfNeeded() {
 					}
 				}
 				catch (Exception e) {
-					this.logger.error(e, "Failed to correct transactional offset(s)");
+					this.logger.error(e, () -> "Failed to correct transactional offset(s): "
+							+ ListenerConsumer.this.lastCommits);
+				}
+				finally {
+					ListenerConsumer.this.lastCommits.clear();
 				}
 			}
 		}

spring-kafka/src/test/java/org/springframework/kafka/listener/TransactionalContainerTests.java

@@ -711,6 +711,7 @@ private void testFixLagGuts(String topic, int whichTm) throws InterruptedExcepti
 		assertThat(latch.await(60, TimeUnit.SECONDS)).isTrue();
 		TopicPartition partition0 = new TopicPartition(topic, 0);
 		assertThat(committed.get().get(partition0).offset()).isEqualTo(2L);
+		assertThat(KafkaTestUtils.getPropertyValue(container, "listenerConsumer.lastCommits", Map.class)).isEmpty();
 		container.stop();
 		pf.destroy();
 	}