Flink: Fix read split table properties being ignored

[vishnuprakaz]

Setting read.split.target-size, read.split.planning-lookback, or read.split.open-file-cost as a table property has no effect on Flink reads. The split options in FlinkReadOptions declare default values, so FlinkConfParser always returns the Flink default and never reaches the table property fallback in FlinkReadConf, even though the docs promise read option, then Flink configuration, then table property. This is a regression from #5967, which introduced the read options with declared defaults; before that the planner only set the split scan options when explicitly configured and the table properties were honored.

The split options now declare no default, like the write options, and the defaults are applied in FlinkReadConf after the table property lookup. Needs a backport to 1.20 and 2.0

[uros-b]

The new test exercises only three of the four precedence tiers (flinkConf > tableProperty > default) for each split option; the top tier; an explicit entry in the readOptions map (the second FlinkReadConf constructor argument) is never asserted. All three tests pass ImmutableMap.of() as the readOptions arg, so the test suite would pass even if the read-option tier were completely broken. Add a fourth assertion per method: instantiate with e.g. ImmutableMap.of(FlinkReadOptions.SPLIT_SIZE, "999") and assert the result beats both the flinkConf value and the table property.

[vishnuprakaz]

Good catch! @uros-b I will cover it.

[vishnuprakaz]

Thanks for the review @uros-b .Added assertions for the read options tier in all three tests.