Jim Hughes

This is looking pretty good. Since implementing FLIP-297 is assigned to @chucheng92, we should get his input as well! @jeyhunkarimov have you signed up for the Flink JIRA? You may...

> Hi @jnh5y could you please check if your comments are addressed? Thanks! Hi @jeyhunkarimov, sorry to be slow to look again. Things are looking good. @dawidwys would you be...

Similar to the table source PR, I peeked at `SinkAbilitySpec`. Looks like `OverwriteSpec`, `PartitioningSpec`, and `WritingMetadataSpec` are covered. I wonder if we should try and cover `RowLevelDeleteSpec` and `RowLevelUpdateSpec`?

@flinkbot run azure

@flinkbot run azure

From reading the code for `StreamExecIntervalJoin`, I noticed pad/filter-left/right transformations. Seems like those can be produced when the `TABLE_EXEC_LEGACY_TRANSFORMATION_UIDS` is used. Since it is a legacy thing, not sure if...

@flinkbot run azure

Hmm... so I took a look at the artifacts here: http://central.maven.org/maven2/org/locationtech/jts/io/jts-io-common/1.15.1/, and the jar doesn't have junit bundled. From a local build of JTS 1.15.1, I don't think any of...

@curtinmartis Any more luck here? If you have any suggestions for how to update/change the JTS poms to make things better for Gradle users, I'm all ears!

@mprins @jjuraszek thanks for looking into this more! From looking briefly, it looks like jts-io-common is pretty much just about GeoJson. Given that lots of projects can read GeoJson in...