Skip to content

Conversation

@welsh-wen
Copy link
Contributor

Purpose of this pull request

Add enable_header_write parameter in FtpFile.md

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@welsh-wen welsh-wen changed the title Wt wendada [Improve][Doc] Add enable_header_write parameter in FtpFile.md Dec 28, 2024
hailin0
hailin0 previously approved these changes Dec 29, 2024
| xml_use_attr_format | boolean | no | - | Only used when file_format is xml. |
| parquet_avro_write_timestamp_as_int96 | boolean | no | false | Only used when file_format is parquet. |
| parquet_avro_write_fixed_as_int96 | array | no | - | Only used when file_format is parquet. |
| enable_header_write | boolean | no | false | Only used when file_format_type is text,csv.<br/> false:don't write header,true:write header. |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| enable_header_write | boolean | no | false | Only used when file_format_type is text,csv.<br/> false:don't write header,true:write header. |
| enable_header_write | boolean | no | false | Only used when file_format_type is text,csv.<br/> `false`: don't write header, `true`: write header. |

@Hisoka-X Hisoka-X changed the title [Improve][Doc] Add enable_header_write parameter in FtpFile.md [Improve][Doc] Add enable_header_write parameter in File doc Dec 30, 2024
@hailin0 hailin0 merged commit 8a187b2 into apache:dev Dec 30, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants