David Radley

In your URL example I think you want to replace the insert in your path as {driverId} is a segment in your url path. Unfortunately at the moment you need...

Looks good, our code is looking to do query params for GET otherwise do body parms. In all cases do path param replacement. And wrap it with a factory with...

@grzegorz8 I am thunking of introducing this as a new factory in [https://github.com/getindata/flink-http-connector/issues/99](https://github.com/getindata/flink-http-connector/issues/99 )

@giulio1979 we have something equivalent but everything is from the Table definition. I hope to ship this factory and enable this sort of templating but config driven rather than the...

I am tracking this in issue http 99 and have a PR out in #149 . I suggest we can close out this issue - as the code will be...

Thanks @rmetzger :-)

> LGTM CI is failing.

@twalthr nit: from my reading of the conversation, it looks like this issue has been fixed in the code already, but we are adding a test, could we update the...

Thanks @X-czh , I see it was agreed to fully drop Kryo v2 support (and hence compatibility from 1.x series). For me it still makes sense to have a migration...

@bjlovegithub please could you prefix the PR title with [BP-2.0] similar to [PR 26561](https://github.com/apache/flink/pull/26561) - it is helpful to easily identify the backports.