flink icon indicating copy to clipboard operation
flink copied to clipboard

[FLINK-28405][Connector/Kafka] Update Confluent Platform images used for testing to v7.1.3

Open MartijnVisser opened this issue 3 years ago • 4 comments

What is the purpose of the change

We have updated the used Kafka Clients to v3.2.1 via FLINK-28060, but we are using Confluent Platform 6.2.2 which contains Kafka 2.8.0.

We should update to Confluent Platform v7.2.1, which includes support for Kafka 3.2.x.

Brief change log

  • Updated all references to Confluent Platform versions (6.2.2) to 7.2.1
  • Synced Zookeeper version again with Kafka 3.2.1 and included a comment with the Kafka version number, so it should be findable next time we update the Kafka Client.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): no
  • The public API, i.e., is any changed class annotated with @Public(Evolving): no
  • The serializers: no
  • The runtime per-record code paths (performance sensitive): no
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
  • The S3 file system connector: no

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

MartijnVisser avatar Jul 05 '22 12:07 MartijnVisser

CI report:

  • e9fc08eac125694c1bb2fc582892bd7c04ef874d Azure: SUCCESS
Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

flinkbot avatar Jul 05 '22 12:07 flinkbot

@flinkbot run azure

MartijnVisser avatar Aug 02 '22 13:08 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Aug 02 '22 13:08 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Aug 02 '22 14:08 MartijnVisser

@zentol Seems I've finally fixed this so would be great if you could have a final look

MartijnVisser avatar Oct 06 '22 03:10 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Oct 07 '22 00:10 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Oct 07 '22 02:10 MartijnVisser

I've ran the failing test multiple times locally, passing every time. Looks like FLINK-24119 but that has been resolved recently.

MartijnVisser avatar Oct 07 '22 02:10 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Oct 09 '22 19:10 MartijnVisser

@flinkbot run azure

MartijnVisser avatar Oct 10 '22 09:10 MartijnVisser

@zentol Can I get one more look at this one too?

MartijnVisser avatar Oct 12 '22 14:10 MartijnVisser