Test `CsvFileFormatParams` instead of `FileFormatParam`

[drmingdrmer]

---

src/meta/proto-conv/tests/it/v075_csv_format_params.rs line 56 at r1 (raw file):

        })
    };
    common::test_load_old(func_name!(), file_format_params_v75.as_slice(), 0, want())?;

FileFormatParams is an enum that does not have version information init(ver and min_reader_ver). Instead of testing decoding FileFormatParam, CsvFileFormatParams is the one should be tested. It contains version information and can be used independently.

Contrastly, FileFormatParam can only be used as part of another protobuf message and should never used alone for the lack of version information and compatibility can not be checked.

Originally posted by @drmingdrmer in https://github.com/datafuselabs/databend/pull/14329#pullrequestreview-1822449681

[drmingdrmer]

This is not fixed IMO.

[bohutang]

Does this PR https://github.com/datafuselabs/databend/pull/14335 needs revert?

[drmingdrmer]

Does this PR #14335 needs revert?

No, #14335 fixed a few other issues, but not this one.