[FLINK-33440] Update Flink version matrix, add 1.19-SNAPSHOT to GH workflows, update flink-connector-parent

[ferenc-csaky]

cc @MartijnVisser

[ferenc-csaky]

I guess we should also update the v3.0 branch, or since there were a couple features added since that would it make sense to create a v3.1 and release that against Flink 1.18?

[snuyanzin]

shoukd we wait a bit since currently 1.16.3 and 1.17.2 are in voting phase and then add all three?

[ferenc-csaky]

shoukd we wait a bit since currently 1.16.3 and 1.17.2 are in voting phase and then add all three?

Yeah, that makes sense. 👍

[ferenc-csaky]

cc @MartijnVisser @snuyanzin

[snuyanzin]

There was snappy update, probably need to add explicit dependency to cope with
https://github.com/apache/flink-connector-hbase/actions/runs/7085132836/job/19876358681?pr=35#step:14:78

[MartijnVisser]

We need to decide how to resolve the Guava dependency convergence: we can't pin the latest version, since that misses classes that are required by HBase. Perhaps we should bump HBase in general?

[ferenc-csaky]

We need to decide how to resolve the Guava dependency convergence: we can't pin the latest version, since that misses classes that are required by HBase. Perhaps we should bump HBase in general?

Sorry for the late answer. There were already a discussion about dropping HBase 1.x back in the day before I started the externalization: https://lists.apache.org/thread/x7l2gj8g93r4v6x6953cyt6jrs8c4r1b

I am okay with dropping it, probably the sooner the better, as it would reduce the complexity of the conenctor a lot, although for 1.18.0 it is not necessary, these dep changes are required for 1.19.

@MartijnVisser @snuyanzin WDYT?

[ferenc-csaky]

@MartijnVisser pinning the older Guava also works, AFAIK externalized connectors are not necessary to depend on the flink-shaded Guava. I am pro removing HBase 1 though, but according to my point that may not blocks this change.

[MartijnVisser]

@ferenc-csaky Will you rebase this PR? I'll close #39

[ferenc-csaky]

@ferenc-csaky Will you rebase this PR? I'll close #39

Yes, will do.

[ferenc-csaky]

@MartijnVisser updated the version matrix, PTAL.

Removed the 1.18.0 run for the v3.0 branch, as it was not released for Flink 1.18. My understanding is that would require a v3.1, right?

[ferenc-csaky]

@snuyanzin I fixed the dep convergence for 1.19 the last CI run failed on, also had to change HBaseTablePlanTest as the JUnit5 migration for TableTestBase in the core repo in 1.19 required some adaptation. Can you retrigger another run pls?