carp84 commented on issue #11622: [FLINK-16913][configuration, statebackend] Migrated StateBackends to use ReadableConfig instead of Configuration URL: https://github.com/apache/flink/pull/11622#issuecomment-608407152 Thanks for the quick response @dawidwys . The touched constructors of `FsStateBackend/MemoryStateBackend` are indeed private so no compatible issue. For `StateBackendFactory#configure` I agree that the impact would be limited, and since `StateBackendFactory` is annotated as `PublicEvolving` I think it's completely ok to change the method signature in major version. I'm just not sure whether it matches our requirement for patch release. It's a little bit off-track since this PR is for master branch (sorry for the disturb), but @aljoscha could you shed some light here? Thanks.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
