Also if we infer the key class and value class for SequenceFiles, should the defaults here be None? The doc and tests imply that we infer them, but we are passing some values here, which is confusing.