OffsetManager func constructRequest() should not set offsetCommitRequestBlock.timestamp when groupInstanceId is not nil

[XinYu52125]

Versions

Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama	Kafka	Go
1.37.2	2.4.2	1.18

Configuration

What configuration values are you using for Sarama and Kafka?

conf.Consumer.Group.InstanceId = someInstanceId
conf.Consumer.Offsets.Retention == 0 // default value

Logs

Non-zero timestamp specified for OffsetCommitRequest not v1, it will be ignored
Non-zero timestamp specified for OffsetCommitRequest not v1, it will be ignored

Problem Description

When calling func constructRequest() with conf.Consumer.Offsets.Retention == 0, OffsetCommitRequest with version 1 will be created.
After offsetCommitRequestBlock is created with perPartitionTimestamp(ReceiveTime), OffsetCommitRequest.version is changed to 7.
Thus, in func (b *offsetCommitRequestBlock) encode, the following check logic will print extra log every time.

	if version == 1 {
		pe.putInt64(b.timestamp)
	} else if b.timestamp != 0 {
		Logger.Println("Non-zero timestamp specified for OffsetCommitRequest not v1, it will be ignored")
	}

[github-actions]

Thank you for taking the time to raise this issue. However, it has not had any activity on it in the past 90 days and will be closed in 30 days if no updates occur.
Please check if the main branch has already resolved the issue since it was raised. If you believe the issue is still valid and you would like input from the maintainers then please comment to ask for it to be reviewed.