GH-2891: Fix isAckAfterHandle with No group.id

Resolves #2891

When using manual partition assignment and `AckMode.MANUAL`, it is possible
to have a `null` `group.id`, whereby Kafka does not maintain any committed offsets.

In this case, we should not attempt to commit the offset after recovery, even if
the error handler `ackAfterHandle` property is true.

**cherry-pick to 3.0.x, 2.9.x**

(cherry picked from commit 9f1050a)

Author: garyrussell

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

@@ -2295,7 +2295,7 @@ private RuntimeException doInvokeBatchListener(final ConsumerRecords<K, V> recor
 				try {
 					this.batchFailed = true;
 					invokeBatchErrorHandler(records, recordList, e);
-					commitOffsetsIfNeeded(records);
+					commitOffsetsIfNeededAfterHandlingError(records);
 				}
 				catch (KafkaException ke) {
 					ke.selfLog(ERROR_HANDLER_THREW_AN_EXCEPTION, this.logger);
@@ -2316,8 +2316,8 @@ private RuntimeException doInvokeBatchListener(final ConsumerRecords<K, V> recor
 			return null;
 		}
 
-		private void commitOffsetsIfNeeded(final ConsumerRecords<K, V> records) {
-			if ((!this.autoCommit && this.commonErrorHandler.isAckAfterHandle())
+		private void commitOffsetsIfNeededAfterHandlingError(final ConsumerRecords<K, V> records) {
+			if ((!this.autoCommit && this.commonErrorHandler.isAckAfterHandle() && this.consumerGroupId != null)
 					|| this.producer != null) {
 				if (this.remainingRecords != null) {
 					ConsumerRecord<K, V> firstUncommitted = this.remainingRecords.iterator().next();
@@ -2781,7 +2781,7 @@ private RuntimeException doInvokeRecordListener(final ConsumerRecord<K, V> cReco
 					}
 					try {
 						invokeErrorHandler(cRecord, iterator, e);
-						commitOffsetsIfNeeded(cRecord);
+						commitOffsetsIfNeededAfterHandlingError(cRecord);
 					}
 					catch (KafkaException ke) {
 						ke.selfLog(ERROR_HANDLER_THREW_AN_EXCEPTION, this.logger);
@@ -2800,8 +2800,8 @@ private RuntimeException doInvokeRecordListener(final ConsumerRecord<K, V> cReco
 			});
 		}
 
-		private void commitOffsetsIfNeeded(final ConsumerRecord<K, V> cRecord) {
-			if ((!this.autoCommit && this.commonErrorHandler.isAckAfterHandle())
+		private void commitOffsetsIfNeededAfterHandlingError(final ConsumerRecord<K, V> cRecord) {
+			if ((!this.autoCommit && this.commonErrorHandler.isAckAfterHandle() && this.consumerGroupId != null)
 					|| this.producer != null) {
 				if (this.isManualAck) {
 					this.commitRecovered = true;