Actually the constructor `FileChecksumMatcher(String checksum, ShardedFile shardedFile)` you suggested already exists in the class as an internal only constructor. Should we deprecate `FileChecksumMatcher(String checksum, ShardedFile shardedFile)`? For other use cases the files to be matched will still be `NumberedShardedFile`. So I think we can keep this constructor as it is.
[ Full content available at: https://github.com/apache/beam/pull/6220 ] This message was relayed via gitbox.apache.org for [email protected]
