Re: [PR] Bump org.yaml:snakeyaml from 1.31 to 2.0 [flink-connector-elasticsearch]

2024-02-13 Thread via GitHub
MartijnVisser commented on PR #90: URL: https://github.com/apache/flink-connector-elasticsearch/pull/90#issuecomment-1941612550 > where did you get this info? Hmmm I would have sworn that was the case, but apparently I was wrong because

Re: [PR] Bump org.yaml:snakeyaml from 1.31 to 2.0 [flink-connector-elasticsearch]

2024-02-13 Thread via GitHub
snuyanzin commented on PR #90: URL: https://github.com/apache/flink-connector-elasticsearch/pull/90#issuecomment-1940820849 @MartijnVisser >since SnakeYAML requires JDK17 where did you get this info? here it is a quote from official SnakeYAML readme [1] >SnakeYAML is a

Re: [PR] Bump org.yaml:snakeyaml from 1.31 to 2.0 [flink-connector-elasticsearch]

2024-02-13 Thread via GitHub
MartijnVisser commented on PR #90: URL: https://github.com/apache/flink-connector-elasticsearch/pull/90#issuecomment-1940775718 @snuyanzin I'm confused, I would have expected CI to fail since SnakeYAML requires JDK17; any idea why everything passed on JDK8 and JDK11? -- This is an