David Radley comments

Results 184 comments of


                                            David Radley

http91 bearer token support

> > @grzegorz8 On the review comment , saying it was not addressed; I do not see any format changes now introduced by the fix. I see : ![Screenshot 2024-10-10...

http91 bearer token support

@grzegorz8 I fixed the format of the one you pointed out and also some in the test files. Let me know if you need anything else changing.

Support Flink 1.20

should we drop 1.16 from the pr build? @kristoffSC @MarekMaj I will look at this once my 2 prs that are currently unmerged are in.

@grzegorz8 @xaniasd @kristoffSC Looking at this in [https://github.com/getindata/flink-http-connector/pull/140](https://github.com/getindata/flink-http-connector/pull/140 ) . Currently the CI is failing so I marked it as draft. As you can see in the PR, I am...

Support Flink 1.20

#140 fixes this

Support Flink 1.20

fyi [https://github.com/getindata/flink-http-connector/issues/143](https://github.com/getindata/flink-http-connector/issues/143)

Allow batch mode to be plain text newline delimited not just Array JSON

@bpb I think it is unusual to have a bespoke non-json , non-XML API that returns CSV records. I suggest you write your own custom format to convert your payload...

Enhance HTTP lookup join to support N:M relationships

@MarekMaj An interesting idea. Is this something you are interested in implementing? It seems to me that it is implicit that we issue a rest call to get one item....

[FLINK-34467] add lineage integration for jdbc connector

@HuangZhenQiu I notice your comment "The PR is not ready for review.", I wonder if you could turn this PR into a draft until it is ready for review -...

[FLINK-34467] add lineage integration for jdbc connector

> Add native lineage support for JDBC connector. It is an ongoing effort. The PR is not ready for review. @HuangZhenQiu It would be worth changing this text in the...